Update selfh.st/icon URLs #1116

Closed
opened 2025-11-20 05:04:42 -05:00 by saavagebueno · 5 comments
Owner

Originally created by @shollyethan on GitHub (May 16, 2025).

Originally assigned to: @MickLesk, @tremor021 on GitHub.

📌 Task summary

Update selfh.st/icon URLs to point to jsDelivr instead of GitHub

📋 Task details

Hey, Community Scripts team! While working through some recent requests I received for icons to be used in this repo, I noticed the current config files for each script are currently pointing to GitHub's own raw file domain:

https://raw.githubusercontent.com/selfhst/icons/refs/heads/main/webp/2fauth.webp

It would be super helpful if the team could update the links to instead use jsDelivr's free CDN service instead:

https://cdn.jsdelivr.net/gh/selfhst/icons/webp/2fauth.webp

This would not only help me track statistics/views, but the icons would also benefit from their global CDN presence.

Thanks!

Originally created by @shollyethan on GitHub (May 16, 2025). Originally assigned to: @MickLesk, @tremor021 on GitHub. ### 📌 Task summary Update selfh.st/icon URLs to point to jsDelivr instead of GitHub ### 📋 Task details Hey, Community Scripts team! While working through some recent requests I received for icons to be used in this repo, I noticed the current config files for each script are currently pointing to GitHub's own raw file domain: https://raw.githubusercontent.com/selfhst/icons/refs/heads/main/webp/2fauth.webp It would be super helpful if the team could update the links to instead use jsDelivr's free CDN service instead: https://cdn.jsdelivr.net/gh/selfhst/icons/webp/2fauth.webp This would not only help me track statistics/views, but the icons would also benefit from their global CDN presence. Thanks!
saavagebueno added the feature label 2025-11-20 05:04:42 -05:00
Author
Owner

@MickLesk commented on GitHub (May 16, 2025):

We can do that, since I've been copying URLs for hours today, that's @tremor021 's job now 😄

@MickLesk commented on GitHub (May 16, 2025): We can do that, since I've been copying URLs for hours today, that's @tremor021 's job now :smile:
Author
Owner

@michelroegl-brunner commented on GitHub (May 16, 2025):

Sure, we are working on updating anything anyway. Should be easy to do with a find/replace!

@michelroegl-brunner commented on GitHub (May 16, 2025): Sure, we are working on updating anything anyway. Should be easy to do with a find/replace!
Author
Owner

@shollyethan commented on GitHub (May 16, 2025):

We can do that, since I've been copying URLs for hours today, that's @tremor021 's job now 😄

Thanks a bunch! I'm getting ready to push ~75% of the icons you requested earlier today -- I should have most of the rest added in the next day or two.

@shollyethan commented on GitHub (May 16, 2025): > We can do that, since I've been copying URLs for hours today, that's [@tremor021](https://github.com/tremor021) 's job now 😄 Thanks a bunch! I'm getting ready to push ~75% of the icons you requested earlier today -- I should have most of the rest added in the next day or two.
Author
Owner

@tremor021 commented on GitHub (May 16, 2025):

We can do that, since I've been copying URLs for hours today, that's @tremor021 's job now 😄

Sure, all the hard lifting is done by me, always...

Anyway, @shollyethan those are great news.

@tremor021 commented on GitHub (May 16, 2025): > We can do that, since I've been copying URLs for hours today, that's [@tremor021](https://github.com/tremor021) 's job now 😄 Sure, all the hard lifting is done by me, always... Anyway, @shollyethan those are great news.
Author
Owner

@tremor021 commented on GitHub (May 17, 2025):

All links use CDN now, as per your instruction @shollyethan. The update is prepared at this branch:
https://github.com/community-scripts/ProxmoxVE/tree/jsons

as soon as big boys approve this, we will push to the main branch

@tremor021 commented on GitHub (May 17, 2025): All links use CDN now, as per your instruction @shollyethan. The update is prepared at this branch: https://github.com/community-scripts/ProxmoxVE/tree/jsons as soon as big boys approve this, we will push to the main branch
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#1116