Skip to content

Conversation

@farhan-nurzi-deriv
Copy link
Contributor

@farhan-nurzi-deriv farhan-nurzi-deriv commented Jan 26, 2024

Changes:

Implement wallet upgrade banner and modal in tradershub.

Screenshots:

Screenshot 2024-01-26 at 9 27 16 AM

[ Updated content ]

Screenshot 2024-01-29 at 3 09 17 PM
Screen.Recording.2024-01-26.at.9.22.12.AM.mov

@vercel
Copy link

vercel bot commented Jan 26, 2024

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

Name Status Preview Updated (UTC)
deriv-app ✅ Ready (Inspect) Visit Preview Feb 7, 2024 7:12am

@github-actions
Copy link
Contributor

github-actions bot commented Jan 26, 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/13149](https://github.com/binary-com/deriv-app/pull/13149)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-fork-farhan-nurzi-deriv-farhan-wall-2063us-422c93.binary.sx?qa_server=red.derivws.com&app_id=24035
    - **Original**: https://deriv-app-git-fork-farhan-nurzi-deriv-farhan-wall-2063us-422c93.binary.sx
- **App ID**: `24035`

@github-actions
Copy link
Contributor

github-actions bot commented Jan 26, 2024

🚨 Lighthouse report for the changes in this PR:

Category Score
🔺 Performance 28
🟧 Accessibility 89
🟢 Best practices 92
🟧 SEO 85
🟧 PWA 78

Lighthouse ran with https://deriv-app-git-fork-farhan-nurzi-deriv-farhan-wall-2063us-422c93.binary.sx/

@coveralls
Copy link

coveralls commented Jan 26, 2024

Coverage Status

coverage: 35.41% (+0.01%) from 35.4%
when pulling a5dc0fd on farhan-nurzi-deriv:farhan/WALL-2063/user-wallet-migration-banner-and-modal
into 37d2705 on binary-com:master.

… farhan/WALL-2063/user-wallet-migration-banner-and-modal
ghost
ghost previously approved these changes Jan 29, 2024
… farhan/WALL-2063/user-wallet-migration-banner-and-modal
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 7, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

16 New issues
0 Security Hotspots
No data about Coverage
4.1% Duplication on New Code

See analysis details on SonarCloud

@nijil-deriv nijil-deriv merged commit 6f15c2b into deriv-com:master Feb 14, 2024
sergei-deriv pushed a commit to sergei-deriv/deriv-app that referenced this pull request Feb 16, 2024
…om#13149)

* feat: ✨ wallet migration banner and modal

* chore: 🧹 remove unused import

* chore: 🧹 add test cases

* chore: ✨ update content

* fix: 🔧 update test

* chore: ✨ add TODO comments

* fix: 🔧 checking active loginid from localstorage instead of authorize hook to prevent extra calls

* refactor: 🔧 styles

* fix: 🔧 using hook from hooks package to prevent logout

* fix: 🔧 logout when new connection open

* fix: 🔧 using localstorage instead of sessionstorage

* fix: 🔧 imports from hooks instead of api

* fix: 🔧 failing tests

* fix: 🔧 incorporate new design

* fix: 🔧 failing tests
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.

5 participants