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

[REFACTORING] shontzu/CFDS-3634/add-compare-accounts-page #126

Conversation

shontzu-deriv
Copy link
Contributor

@shontzu-deriv shontzu-deriv commented Apr 24, 2024

[REFACTORING] Add compare accounts popup as we have in production

a new repository has been initiated for tradershub v2

as the next phase of FE refactoring, tradershub v2 will be deprecated from legacy codebase

we need to migrate all tradershub-v2-related code from legacy codebase to tradershub's new standalone repository

the scope of this card is to migrate the compare-accounts related code

image

@shontzu-deriv shontzu-deriv force-pushed the shontzu/CFDS-3634/add-compare-accounts-page branch from fb6ae2e to 5c288e9 Compare April 24, 2024 08:10
@coveralls
Copy link

coveralls commented Apr 24, 2024

Pull Request Test Coverage Report for Build 8842456181

Details

  • 16 of 63 (25.4%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-1.1%) to 47.34%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/hooks/useCFDAccountsList.tsx 5 15 33.33%
src/hooks/useCFDCompareAccounts.tsx 8 45 17.78%
Totals Coverage Status
Change from base Build 8812886868: -1.1%
Covered Lines: 1390
Relevant Lines: 2367

💛 - Coveralls

@shontzu-deriv shontzu-deriv marked this pull request as ready for review April 24, 2024 08:13
@shontzu-deriv
Copy link
Contributor Author

qa10
appid 1015
[email protected]
Abcd1234!

@shontzu-deriv shontzu-deriv force-pushed the shontzu/CFDS-3634/add-compare-accounts-page branch from 09aaa59 to 05e8363 Compare April 25, 2024 06:37
Copy link

Preview Link: https://pr-126.traders-hub.pages.dev

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

@thisyahlen-deriv thisyahlen-deriv merged commit 00e22b7 into deriv-com:main Apr 26, 2024
10 checks passed
@shontzu-deriv shontzu-deriv changed the title [REFACTORING] [CFDS] [TRAH] shontzu/CFDS-3634/add-compare-accounts-page [REFACTORING] shontzu/CFDS-3634/add-compare-accounts-page May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants