poggit icon indicating copy to clipboard operation
poggit copied to clipboard

Allow external URLs for icons

Open Aericio opened this issue 5 years ago • 6 comments

.poggit.yml declares an icon at https://raw.githubusercontent.com/Aericio/piggydocs-rtd/master/source/_static/img/piggyfactions/icon.png, but there is no such file in your repo! The default icon will be used; your plugin will not be considered for featuring without a custom icon.

I was hoping to move image files off of plugin repositories, but poggit doesn't support URLs. Perhaps you can have a list of allowed domains, such as raw.githubusercontent.com, that poggit would allow.

Aericio avatar Jul 24 '20 07:07 Aericio

Poggit already stores them as a raw.githubusercontent.com URL, Can't see why not, would also mean bigger size'd gifs / images could be used via different repo.

@SOF3 ?

JaxkDev avatar Jul 30 '20 20:07 JaxkDev

If it remains a canonical GitHub file path, I think it's fine. As long as we can ensure user can't change icon after submitting.

SOF3 avatar Jul 31 '20 03:07 SOF3

Any update on this, or should I just add back the image to the repo? Someone brought to my attention that not having an icon hurts our visibility, also causing FactionsPro to gain more downloads than us in the past few months 😅

I would probably have to submit a new version for this if I wanted to add it though.

Aericio avatar Sep 13 '20 13:09 Aericio

I won’t be adding this any time soon, bunch of other things more important to ‘fix’ but I don’t even have time for that right now, unless sofe does something probably won’t be implemented this year lol

JaxkDev avatar Sep 13 '20 13:09 JaxkDev

Alright

Aericio avatar Sep 14 '20 07:09 Aericio

Will look into this next week as I have the week off.

To have a non-changeable icon I think we’d have to only allow raw github url’s with specific commit hash. (We can provide a default url, if we find a valid icon)

JaxkDev avatar Jun 19 '23 14:06 JaxkDev

Adding it into the .poggit.yml

icon: filename/path/url

where URL has to be:

https://raw.githubusercontent.com/USERNAME/REPO/COMMIT_ID/FILE.png

image

JaxkDev avatar May 28 '24 13:05 JaxkDev

Thank you for your work!

Aericio avatar May 29 '24 00:05 Aericio