This allows the community to translate Wikistats into other languages. The base English language strings must be pushed in Wikistats before creating the project in translatewiki.net (T240617). Once that's done I'll update this task tagging translatewiki.net folks to proceed.
Description
Description
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Add support for Wikistats 2.0 | translatewiki | master | +33 -0 | |
| Remove aa.json, replace untranslatable string | analytics/wikistats2 | master | +3 -280 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | • fdans | T238752 Add multilanguage ability to Wikistats | |||
| Resolved | • fdans | T240621 Add stats.wikimedia.org to translatewiki.net |
Event Timeline
Comment Actions
The en.json strings have now been merged to master so they can be added to TranslateWiki:
https://github.com/wikimedia/analytics-wikistats2/blob/master/src/i18n/en.json
I'm also opening a subtask to give access to the repo to l10n-bot
Comment Actions
I assume this https://github.com/wikimedia/analytics-wikistats2/blob/master/src/i18n/aa.json will be removed? aa is a language code for Afar.
I noticed there a string for. This is impossible to translate to many languages that don't use prepositions, especially without message documentation. Please plan adding message documentation for these strings: https://translatewiki.net/wiki/Translating:Localisation_for_developers#Message_documentation
Note for ourselves: we need insertable validators for {0} {foo} and {fooBar} and brace balance validator.
Comment Actions
@Nikerabbit thank you for the quick reply.
I thought I had removed the "for" usage (I only used at the very beginning when learning about i18n templates), but there was an instance still in the code. I'll also remove aa.json in this patch.
Comment Actions
Change 573953 had a related patch set uploaded (by Fdans; owner: Fdans):
[analytics/wikistats2@master] Remove aa.json, replace untranslatable string
Comment Actions
Change 574413 had a related patch set uploaded (by Abijeet Patro; owner: Abijeet Patro):
[translatewiki@master] Add support for wikistats2.0
Comment Actions
I've submitted a patch to add the project to translatewiki.net. Following things need to be checked,
- @fdans - Please let me know if you have a larger icon for the project. This one is too small. This is not necessary, but good to have.
- Please ensure that l10n bot has necessary access.
- Add group description with the key - T:wikistats.
- Please add the qqq.json - message documentation file to the repository.
Comment Actions
https://gerrit.wikimedia.org/r/#/c/analytics/wikistats2/+/573970/ needs to be submitted to achieve that.
Comment Actions
Change 573953 merged by jenkins-bot:
[analytics/wikistats2@master] Remove aa.json, replace untranslatable string
Comment Actions
@abi_ thank you for the list!
Here's the original ring file: https://www.mediawiki.org/wiki/File:WMF_Analytics_-_Ring_Logo_(thick).svg
I'll send a patch addressing the rest of items.
Comment Actions
I've used the logo you provided and updated the patch. I've also added a group description with a default message, please feel free to update it when you can.
Comment Actions
Change 574413 merged by jenkins-bot:
[translatewiki@master] Add support for Wikistats 2.0
Comment Actions
Related patch has been deployed on translatewiki.net and Wikistats 2.0 is now available for translation on twn.
Pending items,
- Update the project page - https://translatewiki.net/wiki/Translating:Wikistats_2.0
- Update the group description - https://translatewiki.net/wiki/Group_descriptions
Comment Actions
Going to close this task as this has been done and translations are being pushed out to the repo. There is an issue with the l10n bot not being able to give CR+2. That is being tracked in T245805: Give access to Wikistats 2 to l10n-bot