Skip to content
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

Fix localization in End of route view #4494

Merged
merged 1 commit into from
Jul 5, 2023

Conversation

S2Ler
Copy link
Contributor

@S2Ler S2Ler commented Jul 5, 2023

And add ja translation for "End Navigation"

Fixes #4492

@S2Ler S2Ler self-assigned this Jul 5, 2023
Add ja translation for "End Navigation"
@S2Ler S2Ler force-pushed the s2/fix-feedback-screen-localization branch from 10fd827 to b96f151 Compare July 5, 2023 06:24
@codecov
Copy link

codecov bot commented Jul 5, 2023

Codecov Report

Merging #4494 (b96f151) into main (9480b33) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4494      +/-   ##
==========================================
- Coverage   59.98%   59.97%   -0.01%     
==========================================
  Files         189      189              
  Lines       21192    21194       +2     
==========================================
  Hits        12712    12712              
- Misses       8480     8482       +2     
Impacted Files Coverage Δ
...es/MapboxNavigation/EndOfRouteViewController.swift 0.00% <0.00%> (ø)

@S2Ler S2Ler marked this pull request as ready for review July 5, 2023 08:00
@S2Ler S2Ler requested a review from a team as a code owner July 5, 2023 08:00
@@ -1,3 +1,6 @@

/* Class = "UILabel"; text = "Rate your trip"; ObjectID = "W5U-cV-cDO"; */
"W5U-cV-cDO.text" = "今回の乗車の評価を行う";

/* Class = "UIButton"; normalTitle = "End Navigation"; ObjectID = "5f2-dT-la4"; */
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't we need to add other localized labels in Japanese?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We localize SDK using transifex service. We will check if it works after latest changes.

@S2Ler S2Ler merged commit 7a15d07 into main Jul 5, 2023
@S2Ler S2Ler deleted the s2/fix-feedback-screen-localization branch July 5, 2023 10:26
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.

[Bug]: Multiple Languages in Feedback
2 participants