Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor(webapp): upgrade node version to 19.4 #5910

Merged
merged 8 commits into from
Feb 2, 2023

Conversation

ulfgebhardt
Copy link
Member

🍰 Pullrequest

upgrade node version to 19.4

Issues

Todo

  • None

@ulfgebhardt ulfgebhardt self-assigned this Jan 28, 2023
@ulfgebhardt ulfgebhardt requested review from Mogge and Tirokk January 28, 2023 03:16
@ulfgebhardt ulfgebhardt added this to the πŸƒ 23/01 January milestone Jan 28, 2023
- Add `"nan": "2.17.0"` to package.json resolutions.
- Add `"nan": "2.17.0"` to package.json resolutions.
Tirokk
Tirokk previously requested changes Jan 28, 2023
Copy link
Member

@Tirokk Tirokk left a comment

Choose a reason for hiding this comment

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

Superb! 😍

Works for me in general!!!
Except the Maintenance I upgraded …

I:

  • upgraded the Node version in the maintenance as well
  • upgraded the Node version in main folder as well
  • added "nan": "2.17.0" to package.json resolutions
  • I tested it on docker with an error for maintenance, see below, but everything else works

Please:

Error on my execution of docker compose up --build:

  • there's an error running docker compose up --build with the maintenance mode. would be nice if you could have a look, see screen shot. old Node version does work for me
  • switching the language does not work for me. was the same before …

Bildschirmfoto 2023-01-28 um 10 45 18

@ulfgebhardt ulfgebhardt merged commit a9e61ae into master Feb 2, 2023
@ulfgebhardt ulfgebhardt deleted the webapp-maintenance-node-19 branch February 2, 2023 17:56
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

πŸ’₯ [DevOps] Wepapp and maintenance – Upgrade Node version to latest now v18.13.0?
3 participants