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.

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

fix 'listincoming' field incoming_capacity_msat. #4913

Merged
merged 1 commit into from
Dec 14, 2021

Conversation

denis2342
Copy link
Contributor

incoming_capacity_msat field showed the value as microsat.

before:

{
"incoming": [
{
"id": "02c75ea34304029851769b13e9ccfddf03166562cc045023d7565aefdd26d74a6c",
"short_channel_id": "704410x1060x1",
"fee_base_msat": "0msat",
"fee_proportional_millionths": 1,
"cltv_expiry_delta": 40,
"incoming_capacity_msat": "11972018176000msat"
},.....

after the patch:

{
"incoming": [
{
"id": "02c75ea34304029851769b13e9ccfddf03166562cc045023d7565aefdd26d74a6c",
"short_channel_id": "704410x1060x1",
"fee_base_msat": "0msat",
"fee_proportional_millionths": 1,
"cltv_expiry_delta": 40,
"incoming_capacity_msat": "11972018176msat"
},.....

@rustyrussell
Copy link
Contributor

Nice! Can you add an 'Changelog-Fixed: JSON-RPC: listincoming showed incoming_capacity_msat field 1000 times actual value.` to the commit msg so it gets picked up for the next release?

Copy link
Collaborator

@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.

utack f01879b

incoming_capacity_msat field showed the value as microsat.

Changelog-Fixed: JSON-RPC: listincoming showed incoming_capacity_msat field 1000 times actual value.
@rustyrussell
Copy link
Contributor

Ack 549af78

Trivial rebase on master to fix flake (I hope!)

@cdecker
Copy link
Member

cdecker commented Dec 6, 2021

Kicked CI again, due to a flake in test_fetchinvoice_3hop

@rustyrussell rustyrussell merged commit 522ee5c into ElementsProject:master Dec 14, 2021
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.

4 participants