-
-
Notifications
You must be signed in to change notification settings - Fork 8.6k
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
Add Polish translations for Jenkins plugin manager #7593
Add Polish translations for Jenkins plugin manager #7593
Conversation
core/src/main/resources/hudson/PluginManager/sidepanel_pl.properties
Outdated
Show resolved
Hide resolved
edab1be
to
329cf46
Compare
Should be fine now. I guess changes were rollback after operation "branch update" on github |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Left some optional cleanup suggestions for your consideration.
@@ -20,6 +20,7 @@ | |||
# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN | |||
# THE SOFTWARE. | |||
Advanced\ Settings=Ustawienia zaawansowane |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
core/src/main/resources/hudson/PluginManager/advanced.jelly
no longer contains an "Advanced Settings" with a capital S after #6783, so this line can be deleted:
Advanced\ Settings=Ustawienia zaawansowane |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then this is technical debt of mentioned PR and should be fixed as follow-up of them. Not by removing Polish translation but removing items from all localization.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed. For this reason, these suggestions were merely optional.
@@ -20,11 +20,14 @@ | |||
# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN | |||
# THE SOFTWARE. | |||
Available\ Plugins=Dostępne wtyczki |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
core/src/main/resources/hudson/PluginManager/available.jelly
no longer contains an "Available Plugins" string with a capital P, so this line can be deleted:
Available\ Plugins=Dostępne wtyczki |
@@ -20,7 +20,10 @@ | |||
# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN | |||
# THE SOFTWARE. | |||
Installed\ Plugins=Zainstalowane wtyczki |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
core/src/main/resources/hudson/PluginManager/installed.jelly
no longer contains an "Installed Plugins" string with a capital I, so this line can be deleted:
Installed\ Plugins=Zainstalowane wtyczki |
Check\ to\ install\ the\ plugin=Zaznacz aby zainstalować wtyczkę | ||
Click\ this\ heading\ to\ sort\ by\ category=Kliknij w nagłówek by posortować po kategorii | ||
Click\ this\ heading\ to\ sort\ by\ category=Kliknij nagłówek, by posortować po kategorii |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This key has been deleted since #4534:
Click\ this\ heading\ to\ sort\ by\ category=Kliknij nagłówek, by posortować po kategorii |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/label ready-for-merge
This PR is now ready for merge, after ~24 hours, we will merge it if there's no negative feedback.
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Testing done
Tested locally on version 2.385
Submitter checklist
@Restricted
or have@since TODO
Javadocs, as appropriate.@Deprecated(since = "TODO")
or@Deprecated(forRemoval = true, since = "TODO")
, if applicable.eval
to ease future introduction of Content Security Policy (CSP) directives (see documentation).Desired reviewers
@jenkinsci/core-pr-reviewers
Maintainer checklist
Before the changes are marked as
ready-for-merge
:upgrade-guide-needed
label is set and there is a Proposed upgrade guidelines section in the pull request title (see example).lts-candidate
to be considered (see query).