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.

Sign up for GitHub

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

Jump to bottom

Ensure that rust-dlc's ChainMonitor is loaded on start-up #1547

Closed
luckysori opened this issue Nov 1, 2023 · 1 comment
Closed

Ensure that rust-dlc's ChainMonitor is loaded on start-up #1547

luckysori opened this issue Nov 1, 2023 · 1 comment
Assignees
@luckysori
Labels
rust-dlc

Comments

@luckysori
Copy link
Contributor

Otherwise the ChainMonitor will be built from scratch and important transactions can be missed on-chain.
@luckysori luckysori self-assigned this Nov 1, 2023
@luckysori luckysori changed the title Ensure that rust-dlcs ChainMonitor is loaded on start-up Ensure that rust-dlc's ChainMonitor is loaded on start-up Nov 1, 2023
@luckysori
Copy link
Contributor Author

Opened p2pderivatives/rust-dlc#166 for this. Will need an approval + another PR in this repo to bring in the changes.

Furthermore, we will need to cater for breaking changes. In fact, we know we have recent breaking changes from a recent update (coming with #1482 I believe) which will cause the new constructor to fail.

@luckysori luckysori mentioned this issue Nov 3, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@luckysori luckysori

Labels
rust-dlc
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@luckysori