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.

Sign up for GitHub

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

Jump to bottom

Payment streams mini-app (AKA payment gadget) #71

Merged
merged 46 commits into from
Nov 16, 2021
Merged

Payment streams mini-app (AKA payment gadget) #71

merged 46 commits into from
Nov 16, 2021

Conversation

gabmontes
Copy link
Member

@gabmontes gabmontes commented Oct 11, 2021 •
edited
Loading

This PR adds a new payments mini-app using the Payment Stream contract.

Related Issue(s)

Closes #57

Screenshots

image

image

image

image

image

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Metrics

Actual effort: 8.5d

@gabmontes gabmontes self-assigned this Oct 11, 2021
@gabmontes gabmontes added the blocked Can't be completed due to external dependency label Oct 11, 2021
@gabmontes gabmontes force-pushed the pf-payments branch 4 times, most recently from 4ed263e to a116f37 Compare October 11, 2021 23:26
@gabmontes gabmontes removed the blocked Can't be completed due to external dependency label Oct 14, 2021
gabmontes and others added 21 commits October 13, 2021 23:09
@gabmontes
Initial payment-stream lib and page
66e71a6
@gabmontes
Update payment-stream lib to new contracts
5f0850a
@gabmontes
Fix PS e2e tests
3414b48
@gabmontes
Add more defaults to the ps lib
ac32130
@gabmontes
Add claim from PS method
22fd9bd
@gabmontes
Add several functions to the PS lib
019b463
@gabmontes
Add PS claimable amount
2073d7e
@gabmontes
Add coverage tests
858d28c
@gabmontes
Include more payment stream data
ae56375
@gabmontes
Approve tokens after craeting a payment stream
557cd37
@gabmontes
Remove extra filter when getting payment streams
a6b3aca
@gabmontes
Update stream payment op gas cost
7e55e55
@gndelia @gabmontes
Add payment-streams utility app
c31416d
@gndelia @gabmontes
Fix real time claimable
f72e71d
@gabmontes
Add PS token rate
2b70e45
@gndelia @gabmontes
Update claimable tokens in tx modal when claiming
dc82f6f
@gabmontes
Fix linting error
c6928b3
@gabmontes
Apply formatting changes
d967eed
@gabmontes
Fix component reference
a1d0b5f
@gabmontes
Update the PS factory contract address and tests
6b0fd30
@gabmontes
Fix code formatting
6f1c12d
@gabmontes gabmontes marked this pull request as ready for review October 14, 2021 02:12
gabmontes and others added 17 commits November 9, 2021 01:11
@gabmontes gabmontes marked this pull request as ready for review November 12, 2021 13:30
@gabmontes gabmontes marked this pull request as draft November 15, 2021 20:59
@gabmontes gabmontes marked this pull request as ready for review November 15, 2021 23:07
@maxivarela maxivarela merged commit 79c0311 into master Nov 16, 2021
@maxivarela maxivarela deleted the pf-payments branch November 16, 2021 15:17
@gabmontes gabmontes linked an issue Nov 16, 2021 that may be closed by this pull request
Update Payment Streams mini-app #76
Closed
@gabmontes gabmontes mentioned this pull request Nov 16, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mbonfoster mbonfoster mbonfoster approved these changes

@maxivarela maxivarela maxivarela approved these changes

@raiseerco raiseerco Awaiting requested review from raiseerco

@gndelia gndelia Awaiting requested review from gndelia

Assignees

@gabmontes gabmontes

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update Payment Streams mini-app Add a "receive payments" mini-app
4 participants
@gabmontes @gndelia @maxivarela @mbonfoster