Skip to content
This repository was archived by the owner on Nov 1, 2022. It is now read-only.

Conversation

@mozilla-l10n-automation-bot
Copy link
Collaborator

n/t

@jonalmeida
Copy link
Contributor

[task 2020-08-21T00:20:23.420Z]   /builds/worker/checkouts/src/components/feature/addons/src/main/res/values-az/strings.xml:152: Error: Inconsistent number of arguments in formatting string mozac_feature_addons_updater_status_successfully_updated; found both 0 and 1 [StringFormatCount]
[task 2020-08-21T00:20:23.420Z]       <string name="mozac_feature_addons_updater_status_successfully_updated">%1$s uğurla yeniləndi</string>
[task 2020-08-21T00:20:23.420Z]       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[task 2020-08-21T00:20:23.420Z]       /builds/worker/checkouts/src/components/feature/addons/src/main/res/values-zh-rTW/strings.xml:172: Conflicting number of arguments here

cc: @Delphine

@Amejia481 Amejia481 requested a review from jonalmeida August 21, 2020 15:05
@Amejia481 Amejia481 added the ✏️ needs changes PRs that need some changes/fixes before they can land label Aug 21, 2020
@Delphine
Copy link
Contributor

Delphine commented Aug 21, 2020

I fixed the error in Pontoon for "az" locale, but I don't know what string is wrong for zh-TW (error part with string name is cut off in your comment above, and I unfortunately don't know how to track it down in taskcluster).
Could you please give me the string name so I can go and fix it as well?
thanks!

@jonalmeida
Copy link
Contributor

@Delphine Doing a quick file search on Github, I found the id that the file values-zh-rTW/strings.xml points to at the line number above: mozac_feature_addons_updater_status_successfully_updated

@mozilla-l10n-automation-bot mozilla-l10n-automation-bot force-pushed the import-l10n branch 2 times, most recently from e8ad641 to 6717136 Compare August 24, 2020 00:04
Copy link
Contributor

@jonalmeida jonalmeida left a comment

Choose a reason for hiding this comment

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

bors r+

@jonalmeida jonalmeida added 🛬 needs landing PRs that are ready to land and removed ✏️ needs changes PRs that need some changes/fixes before they can land labels Aug 25, 2020
@bors
Copy link

bors bot commented Aug 25, 2020

Build succeeded:

@bors bors bot merged commit 24e6c5d into mozilla-mobile:master Aug 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

🛬 needs landing PRs that are ready to land

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants