-
Notifications
You must be signed in to change notification settings - Fork 906
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
msggen: Add decode and decodepay to the mapped bindings #6229
Merged
rustyrussell
merged 12 commits into
ElementsProject:master
from
cdecker:202318-grpc-decodepay
May 5, 2023
Merged
msggen: Add decode and decodepay to the mapped bindings #6229
rustyrussell
merged 12 commits into
ElementsProject:master
from
cdecker:202318-grpc-decodepay
May 5, 2023
Commits on May 3, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 4a2d702 - Browse repository at this point
Copy the full SHA 4a2d702View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0304139 - Browse repository at this point
Copy the full SHA 0304139View commit details -
Configuration menu - View commit details
-
Copy full SHA for 9be978e - Browse repository at this point
Copy the full SHA 9be978eView commit details -
msggen: Disable grpc response -> json response temporarily
We use overrides that omit fields in some cases, which makes the conversion lossy. This also means that until we complete the mapping we can't reconvert back.
Configuration menu - View commit details
-
Copy full SHA for ff8b15a - Browse repository at this point
Copy the full SHA ff8b15aView commit details -
pytest: Add a test for the grpc conversion of listpeerchannels
This is still a huge response, so we better make sure we can actually convert it correctly.
Configuration menu - View commit details
-
Copy full SHA for 69ac062 - Browse repository at this point
Copy the full SHA 69ac062View commit details -
Configuration menu - View commit details
-
Copy full SHA for a2b168b - Browse repository at this point
Copy the full SHA a2b168bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0510537 - Browse repository at this point
Copy the full SHA 0510537View commit details -
Configuration menu - View commit details
-
Copy full SHA for c395cc3 - Browse repository at this point
Copy the full SHA c395cc3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 51eecab - Browse repository at this point
Copy the full SHA 51eecabView commit details -
msggen: Move overrides into the model itself
We were using per-type overrides which caused some asymmetries, where conversions could end up dropping fields as we went along. Essentially each conversion would need to override a superset of the previous one, which then caused issues when attempting to close the loop. By overriding on the model level we ensure that all representations are equivalent and convertible into one another, at the expense of overriding a bit more aggressively, which should be fine anyway.
Configuration menu - View commit details
-
Copy full SHA for b961e23 - Browse repository at this point
Copy the full SHA b961e23View commit details -
Configuration menu - View commit details
-
Copy full SHA for 20b7b8c - Browse repository at this point
Copy the full SHA 20b7b8cView commit details
Commits on May 4, 2023
-
msggen: Add DecodePay to the mappings
Changelog-Added: grpc: Added mapping for `listpeerchannels`, `listclosedchannels`, `decode` and `decodepay` RPC method
Configuration menu - View commit details
-
Copy full SHA for 5dc3a0b - Browse repository at this point
Copy the full SHA 5dc3a0bView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.