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

qtgui: fix i18n #2882

Closed
wants to merge 1 commit into from
Closed

qtgui: fix i18n #2882

wants to merge 1 commit into from

Conversation

markus2330
Copy link
Contributor

Basics

Check relevant points but please do not remove entries.
Do not describe the purpose of this PR in the PR description but:

  • Short descriptions should be in the release notes (added as entry in
    doc/news/_preparation_next_release.md which contains _(my name)_)
    Please always add something to the the release notes.
  • Longer descriptions should be in documentation or in design decisions.
  • Describe details of how you changed the code in commit messages
    (first line should have module: short statement syntax)
  • References to issues, e.g. close #X, should be in the commit messages.

Checklist

Check relevant points but please do not remove entries.
For docu fixes, spell checking, and similar none of these points below
need to be checked.

  • I added unit tests
  • I ran all tests locally and everything went fine
  • affected documentation is fixed
  • I added code comments, logging, and assertions (see Coding Guidelines)
  • meta data is updated (e.g. README.md of plugins and METADATA.ini)

Review

Reviewers will usually check the following:

Labels

  • Add the "work in progress" label if you do not want the PR to be reviewed yet.
  • Add the "ready to merge" label if the build server is happy and also you
    say that everything is ready to be merged.

@markus2330
Copy link
Contributor Author

@darddan: here is an old code that tried to fix the translation. Maybe you can start with this? Do you speak some other language next to german/english? Maybe you can also add one more translation?

@markus2330
Copy link
Contributor Author

@darddan I readded you to ElektraInitiative. Does this PR work for you? Can you rebase and fix it?

1 similar comment
@markus2330
Copy link
Contributor Author

@darddan I readded you to ElektraInitiative. Does this PR work for you? Can you rebase and fix it?

@stale
Copy link

stale bot commented Oct 14, 2022

I mark this stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping by writing a message here or create a new issue/PR with the remainder of this issue/PR.
Thank you for your contributions 💖

@stale stale bot added the stale label Oct 14, 2022
@markus2330 markus2330 closed this Oct 22, 2022
@markus2330 markus2330 deleted the i18n branch October 22, 2022 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant