Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

[CRO] NikitK/CRO-380/tracking issues for partners signup #6363

Merged

Conversation

NikitK-deriv
Copy link
Contributor

@NikitK-deriv NikitK-deriv commented Dec 26, 2023

Changes:

In partner signup form we have tracking system for analytic purpose. In that system we have to fix issue with nulls for partners_signup_error_message field and not valid step_codename.

Include default browser API's pop up which handle when user want to close tab/refresh page

Type of change

  • Bug fix
  • New feature
  • Update feature
  • Refactor code
  • Translation to code
  • Translation to crowdin
  • Script configuration
  • Improve performance
  • Style only
  • Dependency update
  • Documentation update
  • Release

Copy link

vercel bot commented Dec 26, 2023

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

Name Status Preview Updated (UTC)
deriv-com ✅ Ready (Inspect) Visit Preview Jan 10, 2024 9:08am

Copy link
Contributor

Running Lighthouse audit...

@dan-deriv dan-deriv changed the title [CRO] NikitK/cro-380/tracking issues for partners signup [CRO] NikitK/CRO-380/tracking issues for partners signup Dec 26, 2023
@habib-deriv
Copy link
Contributor

@NikitK-deriv
Few smoke test failed. Could you please check if it's related to this PR? Then we'll need a PR to fix the smoke tests as well.

mitra-deriv
mitra-deriv previously approved these changes Jan 3, 2024
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

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

See analysis details on SonarCloud

Copy link
Contributor

Preview Link: https://cro-partners-signup1.deriv-com-preview-links.pages.dev

Name Result
Build status Completed ✅
Preview URL Visit Preview
Action URL Visit Action

Copy link
Contributor

🚀 Smoke test run (2) passed successfully!

Copy link
Contributor

🚀 Smoke test run (3) passed successfully!

Copy link
Contributor

🚀 Smoke test run (4) passed successfully!

Copy link
Contributor

🚀 Smoke test run (1) passed successfully!

@meenakshi-deriv meenakshi-deriv merged commit 331c0a3 into binary-com:master Jan 11, 2024
5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants