-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fix updating error of ress with moduleDetection=false #1619
Conversation
…' into fix-ress-error-with-moduleDetection-force
Visit the preview URL for this PR (updated for commit e8ec09f): https://mobu-waiwai--pr1619-fix-ress-error-with-j5y2lcya.web.app (expires Wed, 04 Dec 2024 17:13:03 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 295e29f2bd6fa9acc70abd1b84aec0de70cff3f6 |
@@ -12,6 +12,7 @@ | |||
name="description" | |||
content="A web application to manage participants and timer for mob-programming!" | |||
/> | |||
<link rel="stylesheet" href="https://unpkg.com/ress/dist/ress.min.css"> |
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.
Drop to depend npm and using unpkg looks simple solution to me.
@@ -2,7 +2,6 @@ import { StrictMode } from "react"; | |||
import { createRoot } from "react-dom/client"; | |||
import App from "./App"; | |||
|
|||
import "ress"; |
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.
Thanks! Looks good solution for me too 👍 |
:goodpoem: |
npm uninstall ress
Closes #1617
I think this is relaetd to following issues