-
Notifications
You must be signed in to change notification settings - Fork 298
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
Farzin/WALL-291/Refactor cashier onboarding into a module #8338
Farzin/WALL-291/Refactor cashier onboarding into a module #8338
Conversation
…n_@deriv/api_package
…e_in_onboarding woth hooks
…usePaymentAgentList hook usage
…n_@deriv/api_package
…n_@deriv/api_package
…n_@deriv/api_package
…n_@deriv/api_package
…n_@deriv/api_package
…n_@deriv/api_package
…n_@deriv/api_package
…ashier_onboarding_to_a_new_route
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we modify this hook to accept account_list
as the input ?
because in packages/hooks/src/useTotalAssetCurrency.ts
line 12 we have the same thing and I was thinking maybe we can use this hook for that. what do you think? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it can be optional 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shayan-deriv Hmmm, Yeah that sounds good 🤔
Let's have a call on this later, I can make the change in the next PR I have which is dependent on this one 🙇🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure man 👍
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
⏳ Generating Lighthouse report... |
Changes:
Please include a summary of the change and which issue is fixed below:
When you need to add unit test
When you need to add integration test
Test coverage checklist (for reviewer)
Type of change