-
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] Wallet - Excluded countries and their currencies are shown #54741
Comments
Triggered auto assignment to @laurenreidexpensify ( |
Triggered auto assignment to @madmax330 ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 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:
|
@madmax330 I don't think this is a real deploy blocker right? |
Agreed 👍 I will help demote |
This comment was marked as outdated.
This comment was marked as outdated.
@MonilBhavsar @joekaufmanexpensify The list looks incomplete here. I am going to include Russia for now. Let me know if we need to make some changes here. |
@madmax330, @laurenreidexpensify Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@madmax330, @laurenreidexpensify Huh... This is 4 days overdue. Who can take care of this? |
Job added to Upwork: https://www.upwork.com/jobs/~021877293134217416084 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat ( |
Umm. This is awkward. But this is already being fixed. |
Can you link the issue, then we can close this? |
ProposalPlease re-state the problem that we are trying to solve in this issue.Some invalid countries and their currencies are displayed when the user selects the bank account. What is the root cause of that problem?This issue is displayed because these countries and currencies information are included the code such as CONST.ts and currencyList.json. What changes do you think we should make in order to solve the problem?These countries information need to delete in the CONST.ts file, and then their currencies information need to delete in the currencyList.json. What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?N/A |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: v9.0.80-2
Reproducible in staging?: Yes
Reproducible in production?: N/A - new feature, doesn't exist in prod
If this was caught during regression testing, add the test name, ID and link from TestRail: #52322
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: Mac 14.5/ Chrome
App Component: User Settings
Action Performed:
Notice IRR, SYP, KPW, CUP, and UAH currencies are shown
Based on Step 4 and 13 on the QA Document
Iran, Cuba, Syria, Ukraine, North Korea are not shown in country selector page and IRR, SYP, KPW, CUP, and UAH currencies are not shown in the currency selector page
Expected Result:
Iran, Cuba, Syria, Ukraine, North Korea are not shown in country selector page and IRR, SYP, KPW, CUP, and UAH currencies are not shown in the currency selector page
Actual Result:
Iran, Cuba, Syria, Ukraine, North Korea are shown in country selector page and IRR, SYP, KPW, CUP, and UAH currencies are shown in the currency selector page
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6704920_1735783957336.Screen_Recording_2025-01-02_at_5.03.20_at_night.mp4
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @parasharrajatUpwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: