-
Notifications
You must be signed in to change notification settings - Fork 299
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/76563/Replace DepositStore
with hooks
#7180
Merged
vinu-deriv
merged 22 commits into
binary-com:develop
from
farzin-deriv:farzin/76563/replace_deposit_store_with_hooks
Jan 17, 2023
Merged
Changes from 7 commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
911fbc8
refactor(cashier): :fire: remove `is_deposit_locked` from `DepositStore`
8ae39aa
refactor(cashier): :fire: remove `submitFundsProtection` from `Deposi…
3deccd7
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
f27d1aa
fix(cashier): :green_heart: fix CI build
2addfb6
fix(cashier): :green_heart: fix CI build
c5212fd
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
7b6ef86
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
047a42c
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
39820c4
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
158df1b
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
a8b726d
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
d16693a
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
0558906
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
0a4d34f
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
b9a31b0
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
0705f63
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
0de99e7
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
5ede719
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
ffdd89d
test(hooks): :white_check_mark: add more test cases for `useDepositLo…
fb90b3f
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
5819466
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
8558804
Merge branch 'develop' into farzin/76563/replace_deposit_store_with_h…
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
is_deposit_lock
fromClientStore
doesn’t have any extra logic butuseDepositLocked
have, I'm not sure if the behaviour is the same or not but since we should have a single source of truth I've changed it to useuseDepositLocked
instead.Please let me know if this is wrong and should be reverted 🙏🏻
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.
@farzin-deriv I believe that it's ok if we will use
useDepositLocked
instead ofis_deposit_lock
fromClientStore
because we showCashierLocked
component first based onis_cashier_locked
status and if we haveis_deposit_locked=true
, we show the user what was wrong with his account (in terms ofis_deposit_lock
status (without checking other extra logic)), and if he fixed hisis_cashier_locked
andis_deposit_locked
to false and then again tries to reach cashier we again checking his deposit status, but now with extra logic as you mentioned and show him an appropriate message. Mmm maybe my explanation was not very clear, but my conclusion is we can replaceis_deposit_lock
fromClientStore
withuseDepositLocked
hook hereThere 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.
@george-usynin-binary Thank you for your explanation buddy 🌹 🙇🏻