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

chore: [UPM-923]/evgeniy/error message for passkey not supported device #14723

Conversation

yauheni-deriv
Copy link
Contributor

Changes:

Added the check for error code to show custom error for not supported device

Copy link

vercel bot commented Apr 22, 2024

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

Name Status Preview Updated (UTC)
deriv-app ✅ Ready (Inspect) Visit Preview Apr 26, 2024 1:22pm

Copy link
Contributor

github-actions bot commented Apr 22, 2024

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/14723](https://github.com/binary-com/deriv-app/pull/14723)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-fork-yauheni-deriv-upm-923add-not-supporte-96766b.binary.sx?qa_server=red.derivws.com&app_id=31387
    - **Original**: https://deriv-app-git-fork-yauheni-deriv-upm-923add-not-supporte-96766b.binary.sx
- **App ID**: `31387`

Copy link
Contributor

github-actions bot commented Apr 22, 2024

🚨 Lighthouse report for the changes in this PR:

Category Score
🔺 Performance 25
🟢 Accessibility 90
🟢 Best practices 92
🟧 SEO 81
🟧 PWA 78

Lighthouse ran with https://deriv-app-git-fork-yauheni-deriv-upm-923add-not-supporte-96766b.binary.sx/

@coveralls
Copy link

coveralls commented Apr 22, 2024

Coverage Status

coverage: 40.421% (+6.3%) from 34.153%
when pulling 46a7bf1 on yauheni-deriv:upm-923_add-not-supported-device-error
into 4da8b11 on binary-com:master.

amina-deriv
amina-deriv previously approved these changes Apr 22, 2024
matin-deriv
matin-deriv previously approved these changes Apr 22, 2024
Copy link

sonarcloud bot commented Apr 26, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.9% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

Generating Lighthouse report...

@amina-deriv amina-deriv merged commit da2049f into binary-com:master Apr 29, 2024
8 checks passed
vinu-deriv pushed a commit to vinu-deriv/deriv-app that referenced this pull request May 28, 2024
…ce (binary-com#14723)

* chore: [UPM-923]/evgeniy/error message for passkey not supported device

* fix: read of undefined

* chore: added new button text and action for not supported

* fix: error variable name

* chore: update content in passkey-config (binary-com#42)

* fix: content for unsupported error

* Suisin/solve passkey header issue (binary-com#43)

* chore: fix header not bold issue

* chore: update passkey creation fail content

* chore: refactor error content

* chore: added comment to remove is_tour_open

* chore: remove close icon for error modal

* chore: remove close icon for error modal

---------

Co-authored-by: Sui Sin <103026762+suisin-deriv@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants