Skip to content

docs(metainfo): add more URLs#4086

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

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

Conversation

@salim-b
Copy link
Copy Markdown
Contributor

@salim-b salim-b commented Jan 22, 2026

@FriedrichFroebel
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
Copy link
Copy Markdown
Contributor Author

salim-b commented Jan 23, 2026

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):

Screenshot 2026-01-23 at 12-22-56 Install Flare on Linux Flathub

Bazaar uses these URLs similarly.

@Mailaender
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

@FriedrichFroebel
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 mgrojo self-requested a review March 22, 2026 18:26
Copy link
Copy Markdown
Member

@mgrojo mgrojo left a comment

Choose a reason for hiding this comment

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

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

@mgrojo mgrojo merged commit 05a92a6 into sqlitebrowser:master Mar 28, 2026
13 checks passed
@mgrojo
Copy link
Copy Markdown
Member

mgrojo commented Mar 28, 2026

Merged. Thanks, @salim-b.

@salim-b salim-b deleted the patch-1 branch March 28, 2026 17:54
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