Skip to content
This repository was archived by the owner on Feb 3, 2025. It is now read-only.

update to latest liquidity lib #959

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

johncantrell97
Copy link
Contributor

  • updates to latest lib
  • needs to use fork to patch ldk back to 118 (same as c=)
  • fixes a bug where we weren't calling peer manager process events, causing delays in message sending

@benthecarman
Copy link
Collaborator

Can't create an invoice when running this, did the token change again?

@johncantrell97
Copy link
Contributor Author

yeah it would need to be live on our node first, we haven't even merged it yet. there's an underlying protocol change in this that both nodes need to have

@benthecarman
Copy link
Collaborator

benthecarman commented Jan 11, 2024

Okay once it is deployed there can you add a note so I can test again

@johncantrell97
Copy link
Contributor Author

This should be deployed to our mutinynet node now

@AnthonyRonning
Copy link
Contributor

AnthonyRonning commented Jan 17, 2024

Okay, we just cut a release to mutiny-node (not mutiny-web) and don't plan on another one for awhile. Is this good to go ahead and merge and cut another release before we freeze mutiny-web for a little bit? Pending any minor fixes discovered during testing.

@johncantrell97
Copy link
Contributor Author

From my perspective it's good to go. I think ben wanted to test it end-to-end just to make sure it works though

Copy link
Collaborator

@benthecarman benthecarman left a comment

Choose a reason for hiding this comment

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

works for me!

@AnthonyRonning AnthonyRonning merged commit 9990697 into MutinyWallet:master Jan 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants