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

[$250] Workspace subscription - Expensify whisper message contains USD instead of GBP #54842

Open
2 of 8 tasks
vincdargento opened this issue Jan 6, 2025 · 6 comments
Open
2 of 8 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review Weekly KSv2

Comments

@vincdargento
Copy link

vincdargento commented Jan 6, 2025

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.

  1. Login with a new public domain user.
  2. Finish onboarding.
  3. Create workspace
    4, Go to Settings > Workspace
  4. More features > turn ON Accounting
  5. Go to Accounting > Connect NetSuit
  6. Upgrade workspace
  7. Go to #admins room

Expected Result:

Whisper message contains GBP currency.

Actual Result:

Whisper message contains USD currency.

Workaround:

Unknown

Platforms:

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

bug.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021876714113440426836
  • Upwork Job ID: 1876714113440426836
  • Last Price Increase: 2025-01-07
Issue OwnerCurrent Issue Owner: @eVoloshchak
@vincdargento vincdargento added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jan 6, 2025
Copy link

melvin-bot bot commented Jan 6, 2025

Triggered auto assignment to @JmillsExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@Tony-MK
Copy link
Contributor

Tony-MK commented Jan 7, 2025

The whisper message comes from api/UpgradeToCorporate? so this needs to be fixed internally.

Screenshot 2025-01-07 at 19 55 21

We also need to fix GenericFeaturesView because it uses workspace.upgrade.commonFeatures.benefits.note which has a static price and the user's local currency will not be displayed.

note: 'The Control plan starts at $9 per active member per month.',

{translate('workspace.upgrade.commonFeatures.benefits.note')}{' '}

Let's follow up on this with PR.

What do you think?

cc: @ahmedGaber93 @amyevans

@amyevans
Copy link
Contributor

amyevans commented Jan 7, 2025

Sounds good. @Tony-MK I'll assign you to work on the frontend portion.

The whisper message comes from api/UpgradeToCorporate? so this needs to be fixed internally.

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:

/**
* Get user's preferred currency in the following order:
*
* 1. Payment card currency
* 2. User's local currency (if it's a valid payment card currency)
* 3. USD (default currency)
*
*/

@amyevans amyevans added Engineering External Added to denote the issue can be worked on by a contributor labels Jan 7, 2025
Copy link

melvin-bot bot commented Jan 7, 2025

Job added to Upwork: https://www.upwork.com/jobs/~021876714113440426836

@melvin-bot melvin-bot bot changed the title Workspace suscription - Expensify whisper message contains USD instead of GBP [$250] Workspace suscription - Expensify whisper message contains USD instead of GBP Jan 7, 2025
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 7, 2025
Copy link

melvin-bot bot commented Jan 7, 2025

Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak (External)

@amyevans amyevans removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 7, 2025
@amyevans amyevans changed the title [$250] Workspace suscription - Expensify whisper message contains USD instead of GBP [$250] Workspace subscription - Expensify whisper message contains USD instead of GBP Jan 7, 2025
@Tony-MK Tony-MK mentioned this issue Jan 7, 2025
50 tasks
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Jan 8, 2025
@Tony-MK
Copy link
Contributor

Tony-MK commented Jan 8, 2025

The PR is ready for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review Weekly KSv2
Projects
None yet
Development

No branches or pull requests

5 participants