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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flatpak Metainfo: Add Affiliation Note #419

Closed
wants to merge 1 commit into from

Conversation

sonic2kk
Copy link
Contributor

Adds the affilication note about the fake website to the Flatpak Metainfo which should also, if my understanding is correct, make it appear on the Flathub page (and possibly elsewhere that can pull this data, such as on the commandline?).

I think I got it right in terms of styling but I haven't ever written anything like this before. I am unsure if the links for example work this way, and if the markup is fairly standard, or if there are some Flatpak (or more specifically, Flathub) specific guidelines in place for excessive linking/linking at all.

@DavidoTek
Copy link
Owner

Thanks!

Adds the affilication note about the fake website to the Flatpak Metainfo which should also, if my understanding is correct,

Hmm. I think it is enough for now that we have added it to the GitHub page. If people found their way to the Flatpak page, there isn't much they can do wrong anymore.

make it appear on the Flathub page (and possibly elsewhere that can pull this data, such as on the commandline?).

It's displayed on the Flathub page and inside the distros app store (GNOME Software, KDE Discover, ...). Not sure if that can also be done using the cli.

I am unsure if the links for example work this way,

The <a></a> tag is unsupported by appstream. I think only plaintext URLs are allowed. I have seen somewhere that even this can be a warning if I remember correctly.


I will close this PR for now as I don't think we absolutely need it at the moment and also only plaintext URLs are allowed in the Flatpak description.

@DavidoTek DavidoTek closed this Jul 27, 2024
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