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

shontzu/WALL-823/inaccurate-jurisdictions-displayed-when-language-is-non-English #8919

Conversation

shontzu-deriv
Copy link
Contributor

@shontzu-deriv shontzu-deriv commented Jun 7, 2023

Changes:

the account_type_card arg has been localized before passing into the switch case, causing it to return default because there are no matching english cases

Screenshots:

@vercel
Copy link

vercel bot commented Jun 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
deriv-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 21, 2023 4:57am

@github-actions
Copy link
Contributor

github-actions bot commented Jun 7, 2023

A production App ID was automatically generated for this PR. (log)

Click here to copy & paste above information.
- **PR**: [https://github.com/binary-com/deriv-app/pull/8919](https://github.com/binary-com/deriv-app/pull/8919)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-fork-shontzu-deriv-shontzu-wall-823inaccur-08ee0d.binary.sx?qa_server=red.binaryws.com&app_id=24190
    - **Original**: https://deriv-app-git-fork-shontzu-deriv-shontzu-wall-823inaccur-08ee0d.binary.sx
- **App ID**: `24190`

@github-actions
Copy link
Contributor

github-actions bot commented Jun 7, 2023

🚨 Lighthouse report for the changes in this PR:

Category Score
🔺 Performance 20
🟧 Accessibility 75
🟢 Best practices 92
🟧 SEO 85
🟢 PWA 90

Lighthouse ran with https://deriv-app-git-fork-shontzu-deriv-shontzu-wall-823inaccur-08ee0d.binary.sx/

@codecov
Copy link

codecov bot commented Jun 7, 2023

Codecov Report

Merging #8919 (9617d93) into master (243b35d) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #8919   +/-   ##
======================================
  Coverage    0.05%   0.05%           
======================================
  Files         117     117           
  Lines        3422    3422           
  Branches      891     891           
======================================
  Hits            2       2           
  Misses       3420    3420           

@shontzu-deriv shontzu-deriv changed the title fix: fixed defaulted switch case caused by translated arg shontzu/WALL-823/inaccurate-jurisdictions-displayed-when-language-is-non-English Jun 8, 2023
hirad-deriv
hirad-deriv previously approved these changes Jun 8, 2023
Comment on lines 118 to 119
case localize('Swap-Free'):
case localize('Swap Free'):
Copy link
Contributor

Choose a reason for hiding this comment

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

Why same condition is used twice?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

because it other languages (i.e. spanish, Indonesian) there is no dash (-) and it end up at default again

@sonarcloud
Copy link

sonarcloud bot commented Jun 21, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
1.2% 1.2% Duplication

@ali-hosseini-deriv
Copy link
Member

✨ PR has been merged by Paimon the Release Bot

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

Successfully merging this pull request may close these issues.

5 participants