-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Docs/installation cdn #4045
Docs/installation cdn #4045
Conversation
Added 3 examples of CDN usage.
Uncommented CDN line
removed Skypack -- not reliable in testing with Codepen.
✅ Deploy Preview for tiptap-embed ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@RicoTrevisan Thank you so much for your contribution. The ESM.RUN sandbox seems to have an error I think. It shows I am wondering how they handle peer dependencies like |
removed esm.run. I couldn't solve the issue.
Hey @svenadlung , thanks for catching that. I've tried to get to the root-cause of that issue, but failed. Let's leave just the esm.sh implementation there. Hopefully this can be of help to someone. Thanks for letting me contribute. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
hi, i am the maintainer of esm.sh, glad to see this PR. if you have any problem with esm.sh please just tell me, i am happy to fix/improve. thanks! |
Please describe your changes
How did you accomplish your changes
I've built examples of each one of them in Codepen:
How have you tested your changes
With the Codepen examples.
How can we verify your changes
Just open the links to the different Codepens to make sure they work.
Remarks
I'm a rookie contributor, I'm trying to learn, but not sure if I'm doing it correctly. If something is missing or unclear, please let me know -- I'd love to learn.
Checklist
Related issues
[add a link to the related issues here]