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

Update lntop bonus guide for RaspiBolt v3 #811

Merged
merged 1 commit into from
Dec 2, 2021
Merged

Update lntop bonus guide for RaspiBolt v3 #811

merged 1 commit into from
Dec 2, 2021

Conversation

kristapsk
Copy link
Collaborator

@kristapsk kristapsk commented Dec 2, 2021 •
edited

What

Fixes guide for RaspiBolt v3. Changes download from armv6 to arm64 and adds instructions about copying necessary LND credentials.

Scope

  • significant change to core configuration
  • independent bonus guide
  • simple bug fix

Note - there is also recent v0.2.0 release of lntop, but it does not provide precompiled binaries.

@Stadicus
Copy link
Collaborator

Stadicus commented Dec 2, 2021 •
edited

Thanks! I also just added #812, which gives the user "admin" access to lnd by default. That should make most additional authentication steps obsolete.

Maybe we can add #812 first into the main guide, and then this one without the additional copy steps after?

@Stadicus Stadicus added the bug Something isn't working label Dec 2, 2021
@Stadicus Stadicus self-assigned this Dec 2, 2021
@kristapsk
Copy link
Collaborator Author

Maybe we can add #812 first into the main guide, and then this one without the additional copy steps after?

I think that's a good idea.

@kristapsk
Copy link
Collaborator Author

Will modify this PR afer #812 is merged.

@Stadicus Stadicus mentioned this pull request Dec 2, 2021
Closed
Stadicus
Stadicus approved these changes Dec 2, 2021
View reviewed changes
Copy link
Collaborator

@Stadicus Stadicus left a comment

Choose a reason for hiding this comment

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

LGTM!

I noticed that v0.2.0 is available, but only the source code. I asked if binary releases are planned. Not a blocker for this pull request, however, can be updated later.

@Stadicus Stadicus merged commit e8ef85f into raspibolt:master Dec 2, 2021
@kristapsk kristapsk deleted the lntop branch December 2, 2021 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Stadicus Stadicus Stadicus approved these changes

Assignees

@Stadicus Stadicus

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@kristapsk @Stadicus