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

786 update node to version 20 #787

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

mikozet
Copy link
Collaborator

@mikozet mikozet commented Jul 4, 2024

Still work in progress...

Update node to version 20 was made. (first commit) with correct compile but error in web browser console.

I have choosen a path to try to resolve it with Webpack update

Update Webpack to version 5 was made (rest commits)

Current state (last commit)
Compiles correctly but with warnings in console terminal as well as in console in browser.
Screens below:
Screenshot from 2024-07-04 16-43-55
Screenshot from 2024-07-04 16-42-13

Steps to consider:
Try to resolve json-loader problem in current state or go back to first commit where only update to node version 20 was made and resolve from there without webpack update..

@mikozet mikozet linked an issue Jul 4, 2024 that may be closed by this pull request
@mikozet mikozet force-pushed the 786-update-node-to-version-20 branch from aa194e7 to 43888e8 Compare August 4, 2024 07:37
Copy link

netlify bot commented Aug 4, 2024

Deploy Preview for preview-testing-circles-garden ready!

Name Link
🔨 Latest commit 43888e8
🔍 Latest deploy log https://app.netlify.com/sites/preview-testing-circles-garden/deploys/66af2fd281aaf600080a92ed
😎 Deploy Preview https://deploy-preview-787--preview-testing-circles-garden.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mikozet
Copy link
Collaborator Author

mikozet commented Aug 7, 2024

Current state:

Repo updated to node version 20 - compiles in terminal correctly. Tested positively functionalities of app without any errors in preview-test environment.

Running tests produces error although:
Screenshot from 2024-08-07 10-48-38

Proposed fix presented here:
CirclesUBI/circles-core#257

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.

Update node to version 20
1 participant