-
Notifications
You must be signed in to change notification settings - Fork 370
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
[wallet] Refactor leftover thunk to sagas #1388
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1388 +/- ##
=========================================
Coverage ? 65.87%
=========================================
Files ? 271
Lines ? 8048
Branches ? 556
=========================================
Hits ? 5302
Misses ? 2632
Partials ? 114
Continue to review full report at Codecov.
|
142f184
to
b5d7b03
Compare
b5d7b03
to
6bd063a
Compare
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.
woot woot woot! no more thunk
try { | ||
Logger.info(TAG, `Writing pending request to database`) | ||
const pendingRequestRef = firebase.database().ref(`pendingRequests`) | ||
return pendingRequestRef.push(paymentInfo) | ||
pendingRequestRef.push(paymentInfo) |
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 returns a ThenableReference (https://firebase.google.com/docs/reference/js/firebase.database.ThenableReference.html), which still has .then and .catch methods - think you should still yield call this imo
eecbe2f
to
9763a05
Compare
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.
Nice work here!! A few optional comments below :)
Also looks like the tests are failing?
…norepo into martinvol/git_rid_thunk
e18ffc7
to
af1dccd
Compare
* master: [Wallet] Wallet can switch between hosted and local node (#1419) [Wallet] Prevent error from Avatar when name is missing (#1454) [Wallet] Show splash screen until JS is ready on iOS (#1453) Use new segment api keys used by both iOS and Android (#1452) [Wallet] Don't log all props, which includes i18n (#1445) [Helm] Updated the helm package to deploy the upgraded blockscout version (#1129) Tiny copy change (#1429) [contractkit] SortedOraclesWrapper + tests (#1405) [wallet] Refactor leftover thunk to sagas (#1388) [Wallet] Fix repeated QR code scanning and related navigation issues (#1439) [Wallet] Show the currency values with correct rounding. (#1435) [Wallet] Fix firebase initialization error on iOS after reinstalling the app (#1423) [Wallet] Use exit on iOS since we can't restart like Android (#1424) [Wallet] Update local currency styles and layout (#1325) Reset pincode cache if unlock fails (#1430)
Description
Removes thunk as a dependency and refactored thunks left as sagas.
Tested
Tests were created. Manual test confirm functionality.
Other changes
Related issues
Backwards compatibility
Everything should work as any reducer changed.