-
Notifications
You must be signed in to change notification settings - Fork 374
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]: Add Payments You've Requested notifications group & screen #1902
Conversation
fea777c
to
e0228ac
Compare
e0228ac
to
2cf50f7
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.
Excellent work! Just some minor comments
packages/mobile/src/paymentRequest/OutgoingPaymentRequestListItem.tsx
Outdated
Show resolved
Hide resolved
packages/mobile/src/paymentRequest/OutgoingPaymentRequestListItem.tsx
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #1902 +/- ##
=========================================
+ Coverage 74.17% 74.47% +0.3%
=========================================
Files 278 281 +3
Lines 7687 7805 +118
Branches 965 682 -283
=========================================
+ Hits 5702 5813 +111
- Misses 1869 1874 +5
- Partials 116 118 +2
Continue to review full report at Codecov.
|
08f5ca2
to
3cd37a1
Compare
* master: (120 commits) Update docker image and instructions (#2000) DOCS correct typos (#1965) Add a space on payment request (#1935) expose signer through getValidator (#1997) Fix validator election migration bug with sorted list insertion (#1998) Deploy an oracle cronjob (#1814) Set default env in attestation service docker image as production (#1999) Attestation Bot POC (#1851) Only allow external RPCs for tx nodes (#1994) Specify web deps properly (#1950) Fix error calculating fees on currencies with high exchange rate (#1937) Small pre-stake-off CLI cleanup (#1953) Migrations: different CLabs groups gets different votes (#1960) [Wallet] Fix broken translation (decline, pay) (#1968) [Wallet]: Add Payments You've Requested notifications group & screen (#1902) [Wallet] Fix iOS bundle script failing with our monorepo setup (#1958) [NotificationService] Change exchange rate stored format (number timestamp and pair) (#1945) [Wallet] Fix premature hiding of syncing banner and increase some timeouts (#1957) Avoid `liner: function not supported in this terminal` (#1955) Support more human readable log output (#1929) ...
Description
incoming
andoutgoing
.Cancel
feature on anoutgoing
payment request.Remind
feature on anoutgoing
payment request.incoming
requests.outgoing
requests.incoming
andoutgoing
requests.Tested
Ran on Android device
Related issues
Backwards compatibility
Yes
Screenshots