Skip to content

Conversation

suisin-deriv
Copy link
Contributor

Changes:

Fix validation issue from account settings personal details.

Screenshots:

Please provide some screenshots of the change.

Copy link

vercel bot commented Jan 18, 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 18, 2024 6:28am

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

@heorhi-deriv heorhi-deriv merged commit b8e8a20 into deriv-com:rc_V20240117_0 Jan 18, 2024
Copy link
Contributor

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/13032](https://github.com/binary-com/deriv-app/pull/13032)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-fork-suisin-deriv-suisin-webrel1903-1905va-208814.binary.sx?qa_server=red.derivws.com&app_id=32134
    - **Original**: https://deriv-app-git-fork-suisin-deriv-suisin-webrel1903-1905va-208814.binary.sx
- **App ID**: `32134`

Copy link
Contributor

🚨 Lighthouse report for the changes in this PR:

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

Lighthouse ran with https://deriv-app-git-fork-suisin-deriv-suisin-webrel1903-1905va-208814.binary.sx/

sandeep-deriv added a commit that referenced this pull request Jan 18, 2024
* Ako/ enable back the wa icon (#13026)

* Revert "Ako/remove whats app icon (#13018)"

This reverts commit 15ba3a6.

* Revert "fix: to disable whatsapp icon from footer (#13011)"

This reverts commit 16152c2.

* [TRAH]/Hamza/trah 2673/compare accounts screen (#13027)

* chore: added compare account screen

* chore: fxed the button not showing + background

* chore: fix mt5 created accounts for demo and real (#13023)

* chore: update validation errors and regex (#13032)

* translations: 📚 sync translations with crowdin (#13034)

Co-authored-by: DerivFE <fe-access@regentmarkets.com>

* ci: add missing env vars (#13039)

---------

Co-authored-by: Ali(Ako) Hosseini <ali.hosseini@deriv.com>
Co-authored-by: Muhammad Hamza <120543468+hamza-deriv@users.noreply.github.com>
Co-authored-by: thisyahlen <104053934+thisyahlen-deriv@users.noreply.github.com>
Co-authored-by: Sui Sin <103026762+suisin-deriv@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: DerivFE <fe-access@regentmarkets.com>
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.

3 participants