Skip to content
/ dash Public
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

Replace UNPKG w/ jsDelivr #3221

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Replace UNPKG w/ jsDelivr #3221

wants to merge 1 commit into from

Conversation

SukkaW
Copy link

@SukkaW SukkaW commented Mar 15, 2025

Start with a description of this PR. Then edit the list below to the items that make sense for your PR scope, and check off the boxes as you go!

Contributor Checklist

  • I have broken down my PR scope into the following TODO tasks
    • task 1
    • task 2
  • I have run the tests locally and they passed. (refer to testing section in contributing)
  • I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR

optionals

  • I have added entry in the CHANGELOG.md
  • If this PR needs a follow-up in dash docs, community thread, I have mentioned the relevant URLS as follows
    • this GitHub #PR number updates the dash docs
    • here is the show and tell thread in Plotly Dash community

UNPKG has not been actively maintained and has been down since Mar 15, 2025 was down from Mar 15, 2025, 2:00 AM and down for 18 hours (unpkg/unpkg#412). Migrating to jsDelivr means a more stable and reliant service.

The PR replaces UNPKG usage with jsDelivr

Sorry, something went wrong.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@SukkaW SukkaW marked this pull request as ready for review March 15, 2025 14:25
@CNFeffery
Copy link
Contributor

It is highly flexible to replace the default unpkg CDN with any other public CDN source for Dash applications. For instance, in the official documentation source code of my fac component library ( https://fac.feffery.tech/ ), I implemented custom modification to switch from unpkg to npmmirror:

https://github.com/CNFeffery/feffery-antd-docs/blob/998d040640c190a87624a3dc897ce846fdbe1c2c/server.py#L14

@SukkaW
Copy link
Author

SukkaW commented Mar 15, 2025

It is highly flexible to replace the default unpkg CDN with any other public CDN source for Dash applications. For instance, in the official documentation source code of my fac component library ( https://fac.feffery.tech/ ), I implemented custom modification to switch from unpkg to npmmirror:

https://github.com/CNFeffery/feffery-antd-docs/blob/998d040640c190a87624a3dc897ce846fdbe1c2c/server.py#L14

Yeah it is possible, but it doesn't hurt to change the default settings so that it would work out of the box.

@gvwilson gvwilson added feature something new P2 considered for next cycle community community contribution labels Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community community contribution feature something new P2 considered for next cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants