-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$250] Workspace subscription - Expensify whisper message contains USD instead of GBP #54842
Comments
Triggered auto assignment to @JmillsExpensify ( |
The whisper message comes from We also need to fix Line 4385 in cd43345
Let's follow up on this with PR. What do you think? |
Sounds good. @Tony-MK I'll assign you to work on the frontend portion.
I can handle this part. For what it's worth there is already some logic in that API command to account for the user's currency, but it is only relying on the billing card's currency, if present, before falling back to USD. I will update it to match the frontend logic of: App/src/hooks/usePreferredCurrency.ts Lines 9 to 16 in cd43345
|
Job added to Upwork: https://www.upwork.com/jobs/~021876714113440426836 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak ( |
The PR is ready for review. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Issue was found while executing QA for PR #54340
Version Number: v9.0.81-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: #54340
Email or phone of affected tester (no customers): N/A
Issue reported by: Applause Internal Team
Device used: MacOS Catalina 10.15.7
App Component: Workspace Settings
Action Performed:
Precondition: use VPN to locate yourself in UK.
4, Go to Settings > Workspace
Expected Result:
Whisper message contains GBP currency.
Actual Result:
Whisper message contains USD currency.
Workaround:
Unknown
Platforms:
Screenshots/Videos
bug.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @eVoloshchakThe text was updated successfully, but these errors were encountered: