Skip to content

Conversation

tlindi
Copy link
Contributor

@tlindi tlindi commented Mar 28, 2025

Description

Adds light weight application which is liked by Lightning community. With albyHub people can use other non-local blockchain depending bitcoin & lnd lightning nodes. I think adding albyHub would raise interrest of many people towards MyNodeBTC.

Uninstall removes albyHub data, which is a bit inconvenient. Also I couldn't get app shown on
Customize Application Versions and front pages either.
I could not install tar.gz from Community apps UI - I get error:

413 Request Entity Too Large
nginx/1.22.1

I could fix those, if there's some docs I'd need to study.

Checklist

  • tested successfully on local MyNode, if yes, list the device(s) below
  • mentioned related open issue, if any, on here

List of test device(s)

Raspi4

Feel free to modify and please comment.

@tehelsper
Copy link
Collaborator

tehelsper commented Mar 28, 2025

Thanks! Did you make the tarball when you tried the community install? It requires a very specific format from the mynode sdk to install right. It's rarely used since most apps are just integrated the way you added this. I'll try to take a closer look soon and include in the v0.3.36 release (v0.3.35 is about to go out).

@tlindi
Copy link
Contributor Author

tlindi commented Mar 28, 2025

Thanks! Did you make the tarball when you tried the community install?

Yes

It requires a very specific format from the mynode sdk to install right.

I used SDK

It's rarely used since most apps are just integrated the way you added this.

Could I deliver you my tar.gz for inspection somehow? Or open issue, or does it matter if this gets into official anyhow?

I'll try to take a closer look soon and include in the v0.3.36 release (v0.3.35 is about to go out).

Thank you very much.

@tlindi tlindi force-pushed the albyhub-0.15.0 branch 2 times, most recently from c787cba to 60c288f Compare March 28, 2025 15:32
@tlindi
Copy link
Contributor Author

tlindi commented Mar 28, 2025

[X] now works for me.

@tlindi tlindi force-pushed the albyhub-0.15.0 branch 3 times, most recently from ec07455 to 5750f3f Compare April 2, 2025 18:05
@tlindi tlindi requested a review from Jacksonearl2468 April 2, 2025 18:07
"name": "GitHub",
"link": "https://github.com/getAlby/hub"
},
"category": "bitcoin_app",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Would suite better to
"category": "lightning_app",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed to lightning_app

@tlindi
Copy link
Contributor Author

tlindi commented Apr 26, 2025

I could change this to Dynamic/Community App, so the MyNodeBTC files would not be touched at all. And also update Hub version too.
Should I just update this PR, or would it be reasonable to open anew one?

"download_type": "source",
"download_source_url": "https://github.com/getAlby/hub/archive/refs/tags/{VERSION}.tar.gz",
"download_binary_url": {
"aarch64": "https://github.com/getAlby/hub/releases/download/{VERSION}/albyhub-Server-Linux-aarch64.tar.bz2",
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can set all these URLs to "unused" or something since the download will be skipped. See jam.json for an example.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

echo $PYBLOCK_VERSION > $PYBLOCK_LATEST_VERSION_FILE
echo $WARDENTERMINAL_VERSION > $WARDENTERMINAL_LATEST_VERSION_FILE
echo $WARDENTERMINAL_VERSION > $WARDENTERMINAL_LATEST_VERSION_FILE
echo $ALBYHUB_VERSION > $ALBYHUB_LATEST_VERSION_FILE
Copy link
Collaborator

Choose a reason for hiding this comment

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

This can be removed. Dynamic apps don't need to go here. I should add a comment in this file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok. Removed addition that is not needed.

systemctl stop albyhub
docker rmi albyhub
docker rmi ghcr.io/getalby/hub:$ALBYHUB_VERSION
rm -rf /mnt/hdd/mynode/albyhub/.albyhub-data
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a reason to delete the data during a reinstall? I would normally add a button to the settings page to clean an apps data. Uninstall / reinstall normally don't clear out storage on the data drive.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree. This removed.

Let's make clean app data button, too.

tlindi added 9 commits May 23, 2025 16:31
Fix albyhub.service: no syslog (btw mynode_sdk .service file needs updateting)
Update albyhub.png - Use GetAlby Brand Kit logo
Need to implement backup
Fix albyhub.service: no syslog (btw mynode_sdk .service file needs updateting)
Update albyhub.png - Use GetAlby Brand Kit logo
Need to implement backup
Dynamics apps json takes care of firewall settings so removing this
@tlindi tlindi force-pushed the albyhub-0.15.0 branch 9 times, most recently from fac0904 to a6f7010 Compare May 23, 2025 15:41
non-used file
@tlindi
Copy link
Contributor Author

tlindi commented May 23, 2025

This works for me

Clear data button is not implemented.

I'd like to implement App Data Management (backup & restore + clear/remove data) onto API separately. And then call (Remove App Data) API from App Info page.

@tehelsper tehelsper merged commit 9c82b3d into mynodebtc:master May 24, 2025
@tlindi tlindi deleted the albyhub-0.15.0 branch May 25, 2025 13:26
@tlindi
Copy link
Contributor Author

tlindi commented May 28, 2025

As reminder, this was requested on ticket
#878

@tlindi tlindi mentioned this pull request Jun 1, 2025
1 task
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