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

Chat - After switching account from Spanish to English the app still shows in Spanish and no option to change language for gmail accounts #5286

Closed
kavimuru opened this issue Sep 16, 2021 · 38 comments
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@kavimuru
Copy link

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Action Performed:

  1. Launch the app
  2. Log in with an account which is Spanish (expensifail domain account)
  3. Log out from account
  4. Log in with an account which is English (Expensifail or gmail account)
  5. Go to Settings
  6. Check the language of the app

Expected Result:

After switch accounts from Spanish to English, all Setting in English language displayed

Actual Result:

After switch accounts from Spanish to English, all Setting in Spanish language displayed. For public domain accounts there is no option to switch the language.

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Version Number: 1.0.99-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Bug5238828_Screen_Recording_20210915-194557_New_Expensify.mp4

Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:

View all open jobs on GitHub

@kavimuru kavimuru added the DeployBlockerCash This issue or pull request should block deployment label Sep 16, 2021
@OSBotify
Copy link
Contributor

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

@MelvinBot
Copy link

Triggered auto assignment to @bondydaa (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@francoisl
Copy link
Contributor

I can reproduce this on production, so I don't think it needs to be a deploy blocker? Going to test again with different accounts to make sure.

@francoisl
Copy link
Contributor

I can reproduce in production, so I'm going to remove the deploy blocker label and we can make this a regular issue to fix.

@francoisl francoisl removed the DeployBlockerCash This issue or pull request should block deployment label Sep 16, 2021
@bondydaa bondydaa removed their assignment Sep 16, 2021
@bondydaa bondydaa added Daily KSv2 and removed Hourly KSv2 labels Sep 16, 2021
@bondydaa
Copy link
Contributor

thanks for taking a look Franc, tossing to the pool

@francoisl francoisl added the External Added to denote the issue can be worked on by a contributor label Sep 16, 2021
@MelvinBot
Copy link

Triggered auto assignment to @Christinadobrzyn (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@Christinadobrzyn
Copy link
Contributor

I think this needed to be paused in favour of focusing on N6 projects. Added N6 hold the label. adding a weekly label to work on this in a week.

Update: talking this over with Kadie, I think I can create the job in Upwork just no PR deployment until after N6.

@Christinadobrzyn
Copy link
Contributor

Hey @bondydaa can I have your opinion if this GH should remain separate from this?

#5113

I also saw this was deployed, do you think it might be causing this issue?

#4507

@isagoico
Copy link

Issue reproducible during KI retests.

@parasharrajat
Copy link
Member

#5019 is a PR and that was already merged.

I have a proposal #5286 (comment) and #5286 (comment)

@kadiealexander
Copy link
Contributor

Please refer to this post for updated information on the n6 hold, we've raised the bonus to $250 for all issues where a PR is created before the N6 hold is lifted.

@Christinadobrzyn
Copy link
Contributor

On hold for n6

@mvtglobally
Copy link

Issue reproducible during KI retests.

1 similar comment
@mvtglobally
Copy link

Issue reproducible during KI retests.

@parasharrajat
Copy link
Member

Proposal
#5286 (comment)

@marcaaron
Copy link
Contributor

Solution looks good 👍

@marcaaron marcaaron removed n6-hold Help Wanted Apply this label when an issue is open to proposals by contributors labels Oct 18, 2021
@Christinadobrzyn
Copy link
Contributor

Hired @parasharrajat for this! 🎉

This job will include a $250 bonus for the n6 hold.

@mvtglobally
Copy link

Issue reproducible during KI retests.

@marcaaron
Copy link
Contributor

Any updates here?

@parasharrajat
Copy link
Member

PR is updated.

@mvtglobally
Copy link

Issue reproducible during KI retests.

@Christinadobrzyn
Copy link
Contributor

not overdue - still working on the PR

@MelvinBot MelvinBot removed the Overdue label Nov 9, 2021
@marcaaron
Copy link
Contributor

What is the status of this issue? It seems like we are working on it in parallel, but I'm not sure. Should it be closed?

Going OOO for the next week so going to unassign myself for now.

@marcaaron marcaaron removed their assignment Nov 12, 2021
@marcaaron
Copy link
Contributor

@nickmurray47 do you know if this is at all different from this issue #6133?

@nickmurray47
Copy link
Contributor

I think they are one and the same. The bug was that we were calling setLocale(), which sets the NVP and onyx key instead of just Onyx.set(ONYXKEYS.NVP_PREFERRED_LOCALE, value). Testing from my PR fixed this bug so I think we should be good once it gets deployed.

@parasharrajat
Copy link
Member

Ok. Thanks for letting us know @nickmurray47. I think we are good to close the PR then. @Christinadobrzyn Please suggest further action for this issue.

@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented Nov 15, 2021

I think they are one and the same. The bug was that we were calling setLocale(), which sets the NVP and onyx key instead of just Onyx.set(ONYXKEYS.NVP_PREFERRED_LOCALE, value). Testing from my PR fixed this bug so I think we should be good once it gets deployed.

@nickmurray47 or @marcaaron - to clarify the next steps for this issue - can you confirm this Upwork job should be closed prior to the PR going to production and @parasharrajat paid out?

@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented Nov 23, 2021

Confirmed with @nickmurray47 to close this job.

Paid @parasharrajat $250 for the proposal and the promised $250 bonus for the offsite hold and closing the job/this post.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
None yet
Development

No branches or pull requests