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

Apply localization to all hard coded text in the Expensify.cash app #3435

Merged
merged 3 commits into from
Jun 9, 2021
Merged

Apply localization to all hard coded text in the Expensify.cash app #3435

merged 3 commits into from
Jun 9, 2021

Conversation

eVoloshchak
Copy link
Contributor

Details

Fixes some issues that were discovered in this pull-request.

  1. this reference is lost in showDocumentPicker
  2. No such key common.settings in languages/en
  3. The error in showDocumentPicker is only printed on the console, it's not intended to be seen by the end user
    and it does not need translated prefix

Fixed Issues

Fixes #2408

@eVoloshchak eVoloshchak requested a review from a team as a code owner June 8, 2021 16:34
@MelvinBot MelvinBot requested review from Jag96 and removed request for a team June 8, 2021 16:34
Copy link
Contributor

@Jag96 Jag96 left a comment

Choose a reason for hiding this comment

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

LGTM

@Jag96 Jag96 merged commit a25a01b into Expensify:main Jun 9, 2021
@OSBotify
Copy link
Contributor

OSBotify commented Jun 9, 2021

🚀 Deployed to staging in version: 1.0.65-4🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@isagoico
Copy link

isagoico commented Jun 9, 2021

Anything that needs to be QAd here? CC @Jag96 @eVoloshchak

@Jag96
Copy link
Contributor

Jag96 commented Jun 9, 2021

Ah no, we can check this off @isagoico!

@OSBotify
Copy link
Contributor

🚀 Deployed to production in version: 1.0.68-4🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Apply localization to all hard coded text in the Expensify.cash app
4 participants