Skip to content

msggen: add low level openchannel methods#7230

Merged
rustyrussell merged 2 commits intoElementsProject:masterfrom
daywalker90:openchannel_low_level
Apr 21, 2024
Merged

msggen: add low level openchannel methods#7230
rustyrussell merged 2 commits intoElementsProject:masterfrom
daywalker90:openchannel_low_level

Conversation

@daywalker90
Copy link
Collaborator

Based on: #7215 + #7217 + #7218

This PR adds the low level openchannel methods:

  • openchannel_abort
  • openchannel_bump
  • openchannel_init
  • openchannel_signed
  • openchannel_update

Aswell as a small fix for methods that contain a '_' like these above.

@daywalker90
Copy link
Collaborator Author

Notice that i changed the amount in the json schema to msat

@cdecker
Copy link
Member

cdecker commented Apr 18, 2024

Rebased on top of #7218 now that it is merged into master 👍

Copy link
Member

@cdecker cdecker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change in denomination in the schema does not match the docs.

@daywalker90
Copy link
Collaborator Author

This one next please: #7238. That CI needs to be running 24/7 or we won't get them all in the 24.05 release

@daywalker90 daywalker90 force-pushed the openchannel_low_level branch from b78ffc5 to 81e77c6 Compare April 20, 2024 09:36
@rustyrussell rustyrussell merged commit fe180e1 into ElementsProject:master Apr 21, 2024
@daywalker90 daywalker90 deleted the openchannel_low_level branch April 22, 2024 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants