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

Fix F-Droid not displaying logo #2079

Merged
merged 1 commit into from
Jul 7, 2024
Merged

Conversation

leoheitmannruiz
Copy link
Contributor

@leoheitmannruiz leoheitmannruiz commented Apr 30, 2024

Description

It seems F-Droid doesn't display Syncthing's logo at their website and in their client (at least on my end it is not shown).

According to the Wayback Machine, F-Droid did show the logo in the past, but it stopped being displayed between October and December of 2022:

https://web.archive.org/web/20221027141012if_/https://f-droid.org/en/packages/com.nutomic.syncthingandroid/
versus
https://web.archive.org/web/20221208220952if_/https://f-droid.org/en/packages/com.nutomic.syncthingandroid/

This PR adds the Syncthing logo to the location expected by F-Droid, as per the Triple T structure which you seem to be using.

Changes

@yurtpage
Copy link

yurtpage commented May 4, 2024

generally speaking it looks like the F-Droid doesn't load the play listings at all.
There are a lot of app description translations here
https://github.com/syncthing/syncthing-android/tree/main/app/src/main/play/listings

But in the F-Droid nothing translated e.g. for the German it's still the English text https://f-droid.org/de/packages/com.nutomic.syncthingandroid/

From the F-Droid article
https://f-droid.org/en/docs/All_About_Descriptions_Graphics_and_Screenshots/

It looks like everything is correct.

Maybe we should ask on the F-Droid forum instead?

@leoheitmannruiz
Copy link
Contributor Author

leoheitmannruiz commented May 5, 2024 via email

@yurtpage
Copy link

yurtpage commented May 5, 2024

Thank you for the detailed response. Please ask F-Droid devs.
Just as guess, maybe the Tripple-T should be triggered to update descriptions but as far I understood the F-Droid can find the descriptions itself

@leoheitmannruiz
Copy link
Contributor Author

Hey @imsodin :)

I assume you are going to bump syncthing-android to 1.27.7 soon.
Could you please take a quick look at this PR before you do so?
Perhaps F-Droid will then display the icon with the next release and the issue is no longer of relevance.

PS: I'm not sure if pinging is appropriate in this context..
I'm very sorry if it's not.
I'd love some feedback in this regard.

@imsodin
Copy link
Member

imsodin commented May 11, 2024

No worries about pinging, I follow everything in this repo anyway. However I am traveling until July with mobile only and won't be doing releases.

@leoheitmannruiz
Copy link
Contributor Author

Ok :)
Enjoy your travels!

Copy link
Member

@imsodin imsodin left a comment

Choose a reason for hiding this comment

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

Thanks, looks good.

Ups, that was a bit more effort than necessary, the logo in that format already existed: https://github.com/syncthing/syncthing/blob/main/assets/logo-512.png :)

@imsodin imsodin changed the title Fix F-Droid not displaying app store logo Fix F-Droid not displaying logo Jul 7, 2024
@imsodin imsodin merged commit 2b31571 into syncthing:main Jul 7, 2024
@leoheitmannruiz
Copy link
Contributor Author

Ups haha, I did look for it :-)
Thanks!

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

3 participants