Skip to content

Comments

Move misplaced strings files#619

Merged
1ec5 merged 2 commits intomasterfrom
1ec5-tx-2017-09-15
Sep 18, 2017
Merged

Move misplaced strings files#619
1ec5 merged 2 commits intomasterfrom
1ec5-tx-2017-09-15

Conversation

@1ec5
Copy link
Contributor

@1ec5 1ec5 commented Sep 16, 2017

Some strings files had been misplaced due to swapped resource slugs on Transifex. As a result, the speech format strings and the feedback UI both had fewer working translations than they should’ve had. I corrected the resource slugs, moved the misplaced files to their correct locations, and updated all strings files based on Transifex. tx skipped some translations of Localizable.strings in MapboxNavigation because recent additions to the source file have pushed the translated rate below 80%.

/ref #505 (review)
/cc @frederoni @bsudekum

Some strings files had been misplaced due to swapped resource slugs on Transifex. These files have been moved to their correct locations, and all strings files have been updated based on Transifex.
@1ec5 1ec5 added bug Something isn’t working topic: localization labels Sep 16, 2017
@1ec5 1ec5 self-assigned this Sep 16, 2017
@1ec5 1ec5 requested review from bsudekum and frederoni September 16, 2017 06:33
@1ec5 1ec5 mentioned this pull request Sep 16, 2017
Copy link
Contributor

@frederoni frederoni left a comment

Choose a reason for hiding this comment

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

Good catch. Should we also pull in the latest from Transifex? There are updates in es,ca,sv, and vi.

@1ec5
Copy link
Contributor Author

1ec5 commented Sep 16, 2017

Should we also pull in the latest from Transifex? There are updates in es,ca,sv, and vi.

I ran tx pull -a but saw no changes. Are you seeing something different?

@frederoni
Copy link
Contributor

Are you seeing something different?

Saw quite a few changes with just tx pull first but now I only get it when forcing tx pull -f

@1ec5
Copy link
Contributor Author

1ec5 commented Sep 18, 2017

Saw quite a few changes with just tx pull first but now I only get it when forcing tx pull -f

You’re right, looks like a number of translations got stuck and only force-pulling worked. No idea why.

@1ec5 1ec5 merged commit a27bf8a into master Sep 18, 2017
@1ec5 1ec5 deleted the 1ec5-tx-2017-09-15 branch September 18, 2017 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants