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

Likhith/trah 3856/add poa document type #16611

Conversation

likhith-deriv
Copy link
Contributor

@likhith-deriv likhith-deriv commented Aug 26, 2024

Changes:

  • Update the description of POA page
  • Included Selected document type dropdown field
  • Populate the dropdown using kyc_auth_status API response
  • Handled form submission
  • Created different layouts for Desktop and Mobile keeping core functionality in the Parent component
  • Implemented POA changes to MT5 in wallets

Copy link

vercel bot commented Aug 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 Sep 30, 2024 11:04am

Copy link
Contributor

github-actions bot commented Aug 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/16611](https://github.com/binary-com/deriv-app/pull/16611)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-fork-likhith-deriv-likhith-trah-3856add-po-f6a42e.binary.sx?qa_server=red.derivws.com&app_id=24087
    - **Original**: https://deriv-app-git-fork-likhith-deriv-likhith-trah-3856add-po-f6a42e.binary.sx
- **App ID**: `24087`

Copy link
Contributor

github-actions bot commented Aug 26, 2024

🚨 Lighthouse report for the changes in this PR:

Category Score
🔺 Performance 26
🟧 Accessibility 70
🟢 Best practices 92
🟧 SEO 77
🟧 PWA 78

Lighthouse ran with undefined

amina-deriv
amina-deriv previously approved these changes Sep 24, 2024
amina-deriv
amina-deriv previously approved these changes Sep 24, 2024
Copy link

sonarcloud bot commented Sep 30, 2024

@amina-deriv amina-deriv merged commit 8bb21d4 into deriv-com:master Oct 3, 2024
8 checks passed
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