Skip to content
This repository has been archived by the owner on Oct 9, 2020. It is now read-only.

Add file extension into the main field. #22

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

Conversation

Xizario
Copy link

@Xizario Xizario commented Jul 16, 2018

Should contains the full name of the main script to avoid errors.

Currently if loaded in npkg as https://unpkg.com/[email protected]
It redirects to https://unpkg.com/[email protected]/json and results in Cannot find "/json" in [email protected]
After the fix it should redirect to https://unpkg.com/[email protected]/json.js and load correctly.

Should contains the full name of the main script to avoid errors.

Currently if loaded in npkg as https://unpkg.com/[email protected]
It redirects to https://unpkg.com/[email protected]/json and results in `Cannot find "/json" in [email protected]`
After the fix it should redirect to `https://unpkg.com/[email protected]/json.js` and load correctly.
@guybedford
Copy link
Member

This seems to be working on the unpkg side now?

@Xizario
Copy link
Author

Xizario commented Sep 1, 2018

@guybedford Looking at their history, seems they fix it the day after the problem happened. Probably lot of other packages failed so they reverted the change. See this.

But for example the issue still happening in jsdelivr
https://cdn.jsdelivr.net/npm/[email protected]/json.js - loads
https://cdn.jsdelivr.net/npm/[email protected] - does not load
https://cdn.jsdelivr.net/npm/systemjs-plugin-json - does not load
image

So it still worth merging it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants