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

fix: docker dev error, json-server version specify #477

Merged
merged 2 commits into from
Feb 15, 2024

Conversation

rockleona
Copy link
Contributor

Types of changes

  • Bugfix

Description

Due to json-server had release a new version 1.0.0-alpha.23,
it must use node18 or higher, since mock-server.Dockerfile did not copy package-lock.json,
so I've edited the line to download the specific version of json-server

Steps to Test This Pull Request

Steps to reproduce the behavior:

  1. run docker-compose -f ./docker-compose-dev.yml up

Expected behavior

  1. Terminal will show the containers mock-server and pycontw-2024-frontend-dev are running
  2. But when access in browser will show window is not defined, it is fined, PR fix: leafleat module import error in ssr mode #474 will fix the error

Related Issue

Related to #471 and #474

@rockleona rockleona requested a review from SivanYeh February 7, 2024 06:23
Copy link

netlify bot commented Feb 7, 2024

👷 Deploy request for ephemeral-sable-89c7e0 pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 2e3139a

@rockleona rockleona requested a review from mattwang44 February 7, 2024 07:52
Copy link
Member

@mattwang44 mattwang44 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

readme 也要改一下?

@rockleona rockleona force-pushed the fix/docker-dev-error branch from 912505e to c5f855c Compare February 15, 2024 01:39
@rockleona
Copy link
Contributor Author

readme 也要改一下?

已經修改了, 我想應該只有 node 版本需要更新?
另外 .nvmrc 也一併修改了

@SivanYeh SivanYeh merged commit 0695ba5 into pycontw:main Feb 15, 2024
1 check passed
@tyuchx
Copy link
Contributor

tyuchx commented Feb 15, 2024

這邊不上到 node 18 or 20 是因為有套件相依性的關係嗎?

#423 似乎在做一樣的事情

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.

4 participants