-
Notifications
You must be signed in to change notification settings - Fork 242
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
Add Lombard Finality Provider #354
Conversation
Hey @fruit37, can you update the PR description to follow this PR template and also check the checkboxes? |
Hey @RafilxTenfen, thank you. I have updated the description.
|
@fruit37 should I reopen the PR? |
@RafilxTenfen, yes, please. Thank you very much. |
@@ -0,0 +1,11 @@ | |||
{ | |||
"description": { | |||
"moniker": "Lombard Finance", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you make sure that the moniker does not contains white spaces
https://github.com/babylonlabs-io/networks/blob/main/bbn-1/finality-providers/README.md?plain=1#L228
After updating, you need to resign and update the Lombard.sig
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
New Lombard Finality Provider
Checklist
guide
Caution
The loss of the (generated keys + mnemonic) makes the finality provider
useless and unable to provide finality, which would lead to no transition to
later phases of the Babylon network.