Skip to content

Migrate to Webpack 5 and Upgrade Node.js to 20 #695

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

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

mmx900
Copy link
Contributor

@mmx900 mmx900 commented Jun 7, 2024

Context

  • Fixes Update app to run on Node 18 or newer (NodeJS 16 has reached End of Support) #682
  • To update Node.js, this PR migrates the project from Webpacker to Webpack 5 using jsbundling-rails. Initially, I also tried Shakapacker, but it seemed to require more effort. Since I am not an expert in Webpack (I primarily use importmap-rails at work), I may not be aware of all the considerations. However, if there is a compelling reason to use Shakapacker, I am open to retrying the migration.
  • While tests are passing, we need to verify that the bundling works correctly on the production server and in other developers' dev environments.

Summary of Changes

Checklist

  • Tested Mobile Responsiveness
  • Added Unit Tests
  • CI Passes
  • Deploys to Heroku on test Correctly (Maintainers will handle)
  • Added Documentation (Service and Code when required)

Screenshots

Before

After

@DeeDeeG
Copy link
Contributor

DeeDeeG commented Jun 15, 2024

Sorry for the delayed response, I will be unavailable to test this until at least the 15th, possible a few days after.

I can spin this up in Docker and see how that goes, and ideally (pay for and run it in) a test Dyno on Heroku.

Thank you for the multiple contributions!

@mmx900
Copy link
Contributor Author

mmx900 commented May 11, 2025

I’m wondering if I could help test this PR.
It looks like there’s a staging server—would it be possible to deploy this branch there and check the logs?
I tried clicking “View logs” at https://github.com/RefugeRestrooms/refugerestrooms/deployments, and it seems to redirect to Heroku, but I don’t have access.

I’m also open to deploying to a Heroku Dyno myself, but since I’m new to Heroku, I wanted to check first if there’s an easier or preferred way.

@DeeDeeG
Copy link
Contributor

DeeDeeG commented Jun 21, 2025

I don't think I can keep maintaining this project "by myself" (de-facto, and for lack of a way to reach the other maintainers to confirm whether help is on the way or not). See: #686 (comment)

Sorry about that.

EDIT to clarify: I don't intend to try and move this or any other PR's forward at this time. I'm retiring from the project. I was already effectively retired, but I left open some hope of the ambiguity of the situation resolving in a positive direction. I don't often hear back from the other maintainers, I don't have the skills by myself. I should simply step away, but I had to come back and say I'm retiring or no-one would know if I am still reviewing PR's or not. So, yeah. I'm retiring from this project. Sorry again.

EDIT 2: Feel free to ping me for information or if I am needed for something along those lines. But I'm not reviewing any new features or fixes. I can't confidently push them to prod anymore, so there would be no benefit to merging them here on GitHub with no way to make it into production.

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 app to run on Node 18 or newer (NodeJS 16 has reached End of Support)
2 participants