-
Notifications
You must be signed in to change notification settings - Fork 11
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
chore(Playroom): upgrade version #1016
Conversation
Size stats
|
Accessibility report ℹ️ You can run this locally by executing |
Deploy preview for mistica-web ready! ✅ Preview Built with commit d9ff546. |
@@ -32,8 +32,8 @@ | |||
"storybook-for-acceptance-tests": "cross-env NODE_ENV=production yarn storybook --ci 2>&1", | |||
"playroom": "playroom start", | |||
"gen-ts-defs": "rimraf dist-ts && tsc --project tsconfig.production.json --emitDeclarationOnly && cp -r dist-ts/src/* dist && rimraf dist-ts", | |||
"vercel-build": "VERCEL_PROD_BUILD=1 yarn storybook-static && ./node_modules/.bin/playroom build && FORCE_MOBILE=1 ./node_modules/.bin/playroom build && FORCE_DESKTOP=1 ./node_modules/.bin/playroom build && cp img/favicon.ico public/", | |||
"vercel-preview-build": "yarn storybook-static && ./node_modules/.bin/playroom build && cp img/favicon.ico public/", | |||
"vercel-build": "VERCEL_PROD_BUILD=1 yarn storybook-static && ./node_modules/.bin/playroom build && FORCE_MOBILE=1 ./node_modules/.bin/playroom build && FORCE_DESKTOP=1 ./node_modules/.bin/playroom build && cp img/favicon.ico public/ && cp img/favicon.svg public/", |
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.
when upgrading storybook to v7, the mistica's favicon stopped working. It seems that storybook replaced the .ico
file with a .svg
.
@@ -107,7 +107,7 @@ const config = { | |||
type: 'asset/resource', | |||
}, | |||
{ | |||
test: /\.vanilla\.css$/i, | |||
test: /\.vanilla\.css/i, |
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.
🎉 This PR is included in version 14.43.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Issue: Link
Remark: it looks like the playroom's site favicon is now the playroom logo instead of Mistica's one. I believe this is caused by this PR in their repo. I've created an issue for them so that they review this.