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 nodejs version #42

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

berbetzl-hmh
Copy link

Why did we do it?
Our goal was to update Panels and its dependencies to Node v16, while cleaning up a little bit.

What did we do?

  • Fix PropTypes, so now we can see launchpad.panels.localhost working with yarn start
  • Dependencies were updated
  • Babel and ESLint configs updated
  • Bundle scripts updated
  • Some exports were updated so no errors would be displayed on code
  • Clean up of unused files

@neil-buckley
Copy link
Collaborator

@berbetzl-hmh great work thank you! This repo isn't maintained any longer, think you're best off keeping your own fork/clone and republishing as your own package. Best of luck, glad to see it still being in use.

@ah3nry
Copy link

ah3nry commented Nov 6, 2023

Hey @neil-buckley and @berbetzl-hmh just seeing this. Looks great. I think forking is a great idea @neil-buckley. We could publish the fork to Artifactory maybe @berbetzl-hmh.

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.

None yet

3 participants