Skip to content

Fix json name mapping for milliSat#57

Merged
vincenzopalazzo merged 2 commits intoclightning4j:masterfrom
hsteinmueller:bug/json_name_mapping
Oct 11, 2021
Merged

Fix json name mapping for milliSat#57
vincenzopalazzo merged 2 commits intoclightning4j:masterfrom
hsteinmueller:bug/json_name_mapping

Conversation

@hsteinmueller
Copy link
Copy Markdown
Contributor

Closes #56, fixed the mapping as described.

Copy link
Copy Markdown
Member

@vincenzopalazzo vincenzopalazzo left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for your amazing work :) and also for the help

I have only a small request.

Comment thread src/main/java/jrpc/clightning/model/types/CLightningChannel.java
Copy link
Copy Markdown
Member

@vincenzopalazzo vincenzopalazzo left a comment

Choose a reason for hiding this comment

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

Thanks for your work!

@vincenzopalazzo review this PR.

@vincenzopalazzo vincenzopalazzo merged commit e2b3944 into clightning4j:master Oct 11, 2021
@hsteinmueller hsteinmueller deleted the bug/json_name_mapping branch October 11, 2021 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fixes mapping json name in ClightningChannels

2 participants