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] [CFDS] [TRAH] Shontzu/cfds 3634/add compare accounts page 2 #128

Conversation

shontzu-deriv
Copy link
Contributor

[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

Copy link

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

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

@coveralls
Copy link

coveralls commented Apr 26, 2024

Pull Request Test Coverage Report for Build 8842528642

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 deleted the shontzu/CFDS-3634/add-compare-accounts-page-2 branch April 26, 2024 03:06
@shontzu-deriv shontzu-deriv restored the shontzu/CFDS-3634/add-compare-accounts-page-2 branch April 26, 2024 03:06
@shontzu-deriv shontzu-deriv reopened this Apr 26, 2024
@shontzu-deriv shontzu-deriv marked this pull request as ready for review April 26, 2024 03:07
@shontzu-deriv
Copy link
Contributor Author

Opened a new PR because the old PR is showing outdated code eventhough changes have been applied:
image
image

@shontzu-deriv shontzu-deriv changed the title Shontzu/cfds 3634/add compare accounts page 2 [REFACTORING] [CFDS] [TRAH] Shontzu/cfds 3634/add compare accounts page 2 Apr 26, 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.

2 participants