Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add translation of dde-shell plugin #141

Merged
merged 2 commits into from
Apr 10, 2024

Conversation

yixinshark
Copy link
Contributor

Add translation of dde-shell plugin

Log: add translation of dde-shell plugin
Influence: translation

cmake/DDEShellPackageMacros.cmake Outdated Show resolved Hide resolved
shell/appletloader.cpp Outdated Show resolved Hide resolved
shell/appletloader.cpp Outdated Show resolved Hide resolved
@@ -86,6 +100,9 @@ DAppletLoader::DAppletLoader(class DApplet *applet, QObject *parent)
{
D_D(DAppletLoader);
d->m_applet = applet;
connect(this, &DAppletLoader::failed, this, [d](const QString &pluginId) {
d->removeTranslation(pluginId);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个对多实例的话,可能失败一个把所有的都卸载了,还不能这样卸载,
不卸载,或者在最后init完做个卸载的检查,

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, yixinshark

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@yixinshark yixinshark force-pushed the feat-ts branch 3 times, most recently from be68f68 to 819417c Compare April 9, 2024 10:31
@deepin-bot
Copy link

deepin-bot bot commented Apr 10, 2024

TAG Bot

New tag: 0.0.7
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #136

Add translation of dde-shell plugin

Log: add translation of dde-shell plugin
Influence: translation
Add translation files

Log: add translation files
Influence: translation
@yixinshark yixinshark merged commit c07a8c1 into linuxdeepin:master Apr 10, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants