Skip to content

Conversation

@hieblmi
Copy link
Collaborator

@hieblmi hieblmi commented Nov 5, 2024

This PR fixes some minor issues in the staging branch of static addr.

  • default 6 deposit confs
  • extract static addr service in its own proto file

func (f *FSM) Errorf(format string, args ...interface{}) {
if f.loopIn == nil {
log.Errorf(format, args...)
log.Infof(format, args...)
Copy link
Member

Choose a reason for hiding this comment

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

Is there a strong reason to change to Infof?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

in the case of error I guess better to keep it. Will change.

@hieblmi
Copy link
Collaborator Author

hieblmi commented Nov 5, 2024

Added another commit to increase the confirmation target from 3 to 6.

Copy link
Member

@bhandras bhandras left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@hieblmi hieblmi merged commit 44092a7 into lightninglabs:static-addr-staging Nov 5, 2024
4 checks passed
@hieblmi hieblmi deleted the static-fixes branch November 5, 2024 20:00
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.

3 participants