Skip to content

Comments

App Submission: Slink#3848

Merged
al-lac merged 67 commits intogetumbrel:masterfrom
dennysubke:slink
Nov 21, 2025
Merged

App Submission: Slink#3848
al-lac merged 67 commits intogetumbrel:masterfrom
dennysubke:slink

Conversation

@dennysubke
Copy link
Contributor

App Submission

Slink

...

Icon

logo

...

Gallery images

1 2 3 4

I have tested my app on:

  • umbrelOS on a Raspberry Pi
  • umbrelOS on an Umbrel Home
  • umbrelOS on Linux VM

Copy link
Contributor

@Mr-Sunglasses Mr-Sunglasses left a comment

Choose a reason for hiding this comment

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

Hey @dennysubke

Thanks a lot for packaging Slink , Really great use case to have this app on Umbrel! 🌟

I truly appreciate your continued contributions and the effort you put into growing the app ecosystem. Amazing work as always! 🚀👏

While testing it on my Umbrel Home, I noticed a few issues that need to be addressed. I’ve added some comments with suggestions for improvement.

@Mr-Sunglasses Mr-Sunglasses added the awaiting changes Waiting for contributor to make requested changes label Oct 23, 2025
@dennysubke
Copy link
Contributor Author

Hey @Mr-Sunglasses!

Thanks for the helpful review! All your points are now resolved. 💪🏻

Copy link
Contributor

@Mr-Sunglasses Mr-Sunglasses left a comment

Choose a reason for hiding this comment

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

Hey @dennysubke 👋🏻,

Thanks for making the changes, really nice work! 🏆

We will now prepare the gallery assets. 🚀

@Mr-Sunglasses Mr-Sunglasses added awaiting gallery assets Umbrel team to make gallery assets and removed awaiting changes Waiting for contributor to make requested changes labels Oct 23, 2025
@github-actions
Copy link

⚠️   Linting finished with 1 warning   ⚠️

Thank you for your submission! This is an automated linter that checks for common issues in pull requests to the Umbrel App Store.

Please review the linting results below and make any necessary changes to your submission.

Linting Results

Severity File Description
⚠️ slink/umbrel-app.yml "icon" and "gallery" needs to be empty for new app submissions:
The "icon" and "gallery" fields must be empty for new app submissions as it is being created by the Umbrel team.

Legend

Symbol Description
Error: This must be resolved before this PR can be merged.
⚠️ Warning: This is highly encouraged to be resolved, but is not strictly mandatory.
ℹ️ Info: This is just for your information.

@al-lac
Copy link
Collaborator

al-lac commented Nov 21, 2025

Hey @dennysubke,

thanks for another amazing app, really great tool to self host 🖼️ 🔝

And thanks @Mr-Sunglasses for reviewing 🥇

I updated it to the latest version and also made it run as user 1000. Tested it one final time and everything works perfectly.

We can go live! 🚀

CleanShot 2025-11-21 at 13 53 04@2x

@al-lac al-lac merged commit 410fd42 into getumbrel:master Nov 21, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting gallery assets Umbrel team to make gallery assets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants