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.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

General fixes #1081

Merged
merged 1 commit into from
May 21, 2024
Merged

General fixes #1081

merged 1 commit into from
May 21, 2024

Conversation

sharknoon
Copy link
Contributor

Hello Umbrel Team,

I have found some issues in some apps:

  • Calibre Web: According to the schema, the website entry is mandatory. Other apps without a website used the GitHub repo as the website. So I have aligned that
  • LN Visualizer: The type of the support entry seems to be a link. All the other apps are using it as a link, therefore I have added a mailto: to the email to be a valid link and be clickable
  • Prowlarr: A missing newline

Thank you so much for your awesome work on UmbrelOS, I am really loving it.
I am exited for future features like https, domain names and backups 馃槏

Kind regards from Germany

Sharknoon

@nmfretz
Copy link
Contributor

nmfretz commented May 21, 2024

Great fixes @sharknoon, thank you very much!

I am exited for future features like https, domain names and backups 馃槏

Coming soon 馃槈

@nmfretz nmfretz merged commit a92875d into getumbrel:master May 21, 2024
@sharknoon
Copy link
Contributor Author

You are welcome 馃槂

@sharknoon sharknoon deleted the general-fixes branch May 21, 2024 05:58
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.

None yet

2 participants