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

Cellxpert continue #5624

Closed
wants to merge 57 commits into from

Conversation

NikitK-deriv
Copy link
Contributor

@NikitK-deriv NikitK-deriv commented Sep 12, 2023

Changes:

Form for affiliate signup registration
Connected with back-end in onSubmit function
Handle different errors like connection lost, username exist
Have a tracking features

[TECHNICAL DEBT]:

  • Developed mostly with styled components so can be refactored with atomic approach in the future
  • Using common useState instead of react-hook-form so can be refactored as well
  • validation is not ideal and should be improved using react-hook-forms
  • we have no updated API for this method so we can see a lot of ts errors here and any in types for that

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

@vercel
Copy link

vercel bot commented Sep 12, 2023

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

Name Status Preview Updated (UTC)
deriv-com ✅ Ready (Inspect) Visit Preview Oct 19, 2023 8:10am

@gitguardian
Copy link

gitguardian bot commented Sep 12, 2023

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id Secret Commit Filename
- Google API Key c691418 src/common/constants.ts View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

@github-actions
Copy link
Contributor

github-actions bot commented Sep 12, 2023

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 0
🟠 Accessibility 88
🟠 Best practices 83
🟢 SEO 100
🟢 PWA 90

Lighthouse ran on https://deriv-com-git-fork-nikitk-deriv-cellxpert-continue.binary.sx/

@github-actions
Copy link
Contributor

🚀 Smoke test run (5) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (2) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (4) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (3) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (1) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (2) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (1) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (4) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (5) passed successfully!

@github-actions
Copy link
Contributor

❌ Smoke test run (1) failed. See logs for details: Visit Action

@github-actions
Copy link
Contributor

❌ Smoke test run (3) failed. See logs for details: Visit Action

@github-actions
Copy link
Contributor

❌ Smoke test run (5) failed. See logs for details: Visit Action

@github-actions
Copy link
Contributor

🚀 Smoke test run (3) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (4) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (2) passed successfully!

@github-actions
Copy link
Contributor

🚀 Smoke test run (1) passed successfully!

@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot E 1 Security Hotspot
Code Smell A 5 Code Smells

No Coverage information No Coverage information
3.4% 3.4% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@github-actions
Copy link
Contributor

❌ Smoke test run (4) failed. See logs for details: Visit Action

@github-actions
Copy link
Contributor

❌ Smoke test run (2) failed. See logs for details: Visit Action

@github-actions
Copy link
Contributor

❌ Smoke test run (5) failed. See logs for details: Visit Action

@github-actions
Copy link
Contributor

🚀 Smoke test run (3) passed successfully!

@github-actions
Copy link
Contributor

Preview Link: https://cellxpert-continue.deriv-com-preview-links.pages.dev

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

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.

5 participants