Skip to content
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

Update deps in web #292

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Update deps in web #292

wants to merge 5 commits into from

Conversation

MurakamiShinyu
Copy link
Member

Update next 12->14, react 17->18, etc.

Copy link

vercel bot commented Nov 10, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
vivliostyle-pub-develop ✅ Ready (Inspect) Visit Preview Nov 10, 2023 9:56am

@MurakamiShinyu
Copy link
Member Author

next 12->14, react 17->18 など更新を行い、エラーが起きないようソースコードの修正を行っている。

しかし、 yarn dev でのテスト実行 http://localhost:9990 で編集するリポジトリを開くと、いろいろ不具合が:

  • 「テーマを変更しました: plain-theme」のメッセージがいくつも繰り返し出る
    スクリーンショット 2023-11-10 19 06 33

  • 「Save document」ボタンがファイルを編集したかどうかに関係なく常に有効になる

  • 「Save document」をクリックして保存しようとするとエラーメッセージ「ファイル(manuscript.md)の保存に失敗しました。: status code 400」が出る。GitHubでリポジトリを確認すると実際には保存できている。
    スクリーンショット 2023-11-10 19 08 14

  • ブラウザのコンソールに次のエラー

Warning: Cannot update a component (`LogBufferContextProvider`) while rendering a different component (`CurrentThemeContextProvider`). To locate the bad setState() call inside `CurrentThemeContextProvider`, follow the stack trace as described in https://reactjs.org/link/setstate-in-render
    at CurrentThemeContextProvider (webpack-internal:///./src/middlewares/contexts/useCurrentThemeContext.tsx:75:26)
    at div
    at eval (webpack-internal:///./node_modules/@emotion/react/dist/emotion-element-c39617d8.browser.esm.js:60:66)
    at ChakraComponent (webpack-internal:///./node_modules/@chakra-ui/system/dist/chunk-5PL47M24.mjs:50:102)
    at GitHubOwnerRepo (webpack-internal:///./src/pages/github/[owner]/[repo].tsx:56:72)
    at div
    at eval (webpack-internal:///./node_modules/@emotion/react/dist/emotion-element-c39617d8.browser.esm.js:60:66)
    at ChakraComponent (webpack-internal:///./node_modules/@chakra-ui/system/dist/chunk-5PL47M24.mjs:50:102)
    at AppContextProvider (webpack-internal:///./src/middlewares/contexts/useAppContext.tsx:212:26)
    at LogContextProvider (webpack-internal:///./src/middlewares/contexts/useLogContext.tsx:95:26)
    at LogBufferContextProvider (webpack-internal:///./src/middlewares/contexts/useLogContext.tsx:44:26)
    at EnvironmentProvider (webpack-internal:///./node_modules/@chakra-ui/react-env/dist/chunk-VMD3UMGK.mjs:34:11)
    at ColorModeProvider (webpack-internal:///./node_modules/@chakra-ui/color-mode/dist/chunk-AMBGAKG2.mjs:29:5)
    at ThemeProvider (webpack-internal:///./node_modules/@emotion/react/dist/emotion-element-c39617d8.browser.esm.js:123:50)
    at ThemeProvider (webpack-internal:///./node_modules/@chakra-ui/system/dist/chunk-MFVQSVQB.mjs:28:11)
    at ChakraProvider (webpack-internal:///./node_modules/@chakra-ui/provider/dist/chunk-3DDHO3UN.mjs:20:5)
    at ChakraProvider2 (webpack-internal:///./node_modules/@chakra-ui/react/dist/chunk-QAITB7GG.mjs:19:5)
    at MyApp (webpack-internal:///./src/pages/_app.tsx:45:27)
    at PathnameContextProviderAdapter (webpack-internal:///./node_modules/next/dist/shared/lib/router/adapters.js:100:26)
    at ErrorBoundary (webpack-internal:///./node_modules/next/dist/compiled/@next/react-dev-overlay/dist/client.js:2:5391)
    at ReactDevOverlay (webpack-internal:///./node_modules/next/dist/compiled/@next/react-dev-overlay/dist/client.js:2:7787)
    at Container (webpack-internal:///./node_modules/next/dist/client/index.js:95:27)
    at AppContainer (webpack-internal:///./node_modules/next/dist/client/index.js:237:26)
    at Root (webpack-internal:///./node_modules/next/dist/client/index.js:473:27)

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

Successfully merging this pull request may close these issues.

1 participant