-
Notifications
You must be signed in to change notification settings - Fork 687
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
Updating weblate reference codes. #3935
Updating weblate reference codes. #3935
Conversation
edec77c
to
d3ab065
Compare
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.
Requires updates to the screenshots. Also, do not update the *.po
files here. Even before this one, we have to make sure that we can use/merge the development back into the weblate fork properly.
cca931e
to
afc08dc
Compare
@kushaldas Thanks for reviewing this. I have updated the files according to your comment. I am apologized if I have increased troubles. Have a great day! |
securedrop/i18n_tool.py
Outdated
@@ -26,7 +26,7 @@ class I18NTool(object): | |||
# | |||
# The database of support language, indexed by the language code | |||
# used by weblate (i.e. whatever shows as CODE in | |||
# https://weblate.securedrop.club/projects/securedrop/securedrop/CODE/ | |||
# https://github.com/freedomofpress/securedrop-i18n/tree/i18n/securedrop/translations |
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.
Just change the .club URL to the .org URL here.
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.
Please make that change, I will push the rest of the changes on top of this.
Also do not say that this fixes #3929, we sill have to change other things on top of this. You can remove that line from the comment and say this is a partial fix. |
Partialy solves: freedomofpress#3929
afc08dc
to
83c930c
Compare
@kushaldas I have updated the PR description and the code according to your comments. Please review them again. 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.
Approved, I will test and update the rest.
Update docs to account for Weblate
Status
Completed
Description of Changes
Partially fixes #3929
Testing
Checklist
If you made changes to documentation:
make docs-lint
) passed locally