FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

docs(metainfo): add more URLs by salim-b · Pull Request #4086 · sqlitebrowser/sqlitebrowser · GitHub

docs(metainfo): add more URLs - #4086

Merged
mgrojo merged 1 commit into
sqlitebrowser:masterfrom
salim-b:patch-1
Mar 28, 2026
Merged

docs(metainfo): add more URLs#4086
mgrojo merged 1 commit into
sqlitebrowser:masterfrom
salim-b:patch-1

Conversation

salim-b commented Jan 22, 2026

Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

Thanks for the PR. Could you please elaborate why you consider these URLs to be relevant here? According to the docs, only the homepage is mandatory and the VCS URL is recommended.

salim-b commented Jan 23, 2026

Copy link
Copy Markdown
Contributor Author

The URLs defined here in the AppStream metadata are i.a. used by package managers to provide direct navigation hints to the respective resources.

If the type="donation" URL is set, Flathub displays a prominent Donate button next to the Install button at the top right (example: Clapgrep).

The rest of the URLs are displayed in a dedicated Links block (example: Flare):

Bazaar uses these URLs similarly.

Copy link
Copy Markdown
Contributor

The link back to GitHub also resolves a linter warning at @flathub which they claim may become an error at any time. https://docs.flathub.org/docs/for-app-authors/linter#appstream-missing-vcs-browser-url

Copy link
Copy Markdown
Contributor

I do not feel safe enough here to decide whether all of these values make sense for DB4S. Thus wait for a second opinion from the team here.

mgrojo self-requested a review March 22, 2026 18:26

mgrojo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I agree with the changes. If nobody oppose, I'll merge it in some days.

mgrojo merged commit 05a92a6 into sqlitebrowser:master Mar 28, 2026
13 checks passed

mgrojo commented Mar 28, 2026

Copy link
Copy Markdown
Member

Merged. Thanks, @salim-b.

salim-b deleted the patch-1 branch March 28, 2026 17:54
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL