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

itest: switch over to routerrpc.SendPaymentV2 #4132

Merged
merged 2 commits into from
May 22, 2020
Merged

itest: switch over to routerrpc.SendPaymentV2 #4132

merged 2 commits into from
May 22, 2020

Conversation

joostjager
Copy link
Contributor

No description provided.

@joostjager joostjager mentioned this pull request Mar 31, 2020
Merged
@joostjager joostjager force-pushed the itests-on-routerrpc branch 2 times, most recently from 6f158ab to 056f989 Compare April 1, 2020 06:17
@Roasbeef Roasbeef added this to the 0.11.0 milestone Apr 14, 2020
@Roasbeef Roasbeef added v0.11 itests Issues related to integration tests. labels Apr 14, 2020
@joostjager joostjager force-pushed the itests-on-routerrpc branch 2 times, most recently from 5ef66eb to 574499d Compare May 6, 2020 13:01
@joostjager joostjager changed the title itest: switch over to routerrpc.SendPayment itest: switch over to routerrpc.SendPaymentV2 May 6, 2020
@joostjager joostjager force-pushed the itests-on-routerrpc branch 3 times, most recently from 79275f0 to 3867226 Compare May 7, 2020 06:41
@joostjager joostjager requested a review from wpaulino May 13, 2020 18:38
@joostjager
Copy link
Contributor Author

@wpaulino notes there are more that don't use V2

@joostjager joostjager force-pushed the itests-on-routerrpc branch 2 times, most recently from 9c026e5 to 40c4788 Compare May 15, 2020 10:10
@joostjager
Copy link
Contributor Author

Remaining routerrpc.SendPayment calls converted to routerrpc.SendPaymentV2

@joostjager joostjager merged commit a8d5c32 into lightningnetwork:master May 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wpaulino wpaulino wpaulino approved these changes

@cfromknecht cfromknecht cfromknecht approved these changes

Assignees
No one assigned
Labels
itests Issues related to integration tests.
Projects
None yet
Milestone
0.11.0
Development

Successfully merging this pull request may close these issues.
4 participants
@joostjager @cfromknecht @wpaulino @Roasbeef