Skip to content

Conversation

@likhith-deriv
Copy link
Contributor

Changes:

For fully authenticated clients, POA is being asked upon creating MT5 regulated account
POA should not be asked when CR account is fully authenticated

@boring-cyborg boring-cyborg bot added the Shared label Jan 12, 2024
@vercel
Copy link

vercel bot commented Jan 12, 2024

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

Name Status Preview Updated (UTC)
deriv-app ✅ Ready (Inspect) Visit Preview Jan 12, 2024 0:07am

@likhith-deriv likhith-deriv changed the title fix: poa trigger issue [COJ]/likhith/WEBREL-1894/fix: poa trigger issue Jan 12, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Jan 12, 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/12930](https://github.com/binary-com/deriv-app/pull/12930)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-fork-likhith-deriv-likhith-webrel-1894fix-8ff8c0.binary.sx?qa_server=red.derivws.com&app_id=23879
    - **Original**: https://deriv-app-git-fork-likhith-deriv-likhith-webrel-1894fix-8ff8c0.binary.sx
- **App ID**: `23879`

@github-actions
Copy link
Contributor

github-actions bot commented Jan 12, 2024

🚨 Lighthouse report for the changes in this PR:

Category Score
🔺 Performance 21
🟧 Accessibility 89
🟢 Best practices 92
🟢 SEO 92
🟧 PWA 80

Lighthouse ran with https://deriv-app-git-fork-likhith-deriv-likhith-webrel-1894fix-8ff8c0.binary.sx/

@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@likhith-deriv likhith-deriv merged commit 111a8d5 into deriv-com:rc_V20240111_0 Jan 12, 2024
likhith-deriv added a commit that referenced this pull request Jan 12, 2024
* fix: poa trigger issue

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

2 participants