Skip to content
This repository has been archived by the owner on Dec 11, 2022. It is now read-only.

Update to Meteor 2.0 and enable HMR #45

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

Conversation

loan-laux
Copy link
Contributor

Impact: minor
Type: performance|chore

Issue

Meteor was outdated as version 2.0 recently came out. This version introduces HMR with React Fast Refresh, making for a better developer experience when working on the front-end.

Solution

Update to Meteor 2.0 and enable HMR.

Breaking changes

None.

Testing

  1. Start reaction-identity, check that the UI is working as it should.
  2. Make a change to a React component.
  3. Check that the change takes effect through HMR rather than a page reload.

@loan-laux
Copy link
Contributor Author

Let's wait until the reactioncommerce/meteor:2.0.0-v1 Docker image is available. I'll update the Dockerfile for reaction-identity then.

@Akarshit
Copy link
Contributor

Akarshit commented Aug 4, 2021

@loan-laux Do you still want to keep this around?

@loan-laux
Copy link
Contributor Author

@Akarshit I'll resolve the conflicts and then we should merge it. No harm in keeping reaction-identity up to date even though it'll be deprecated soon.

@loan-laux loan-laux force-pushed the loan-update-meteor-2.0 branch from d665d90 to a5d57ef Compare August 12, 2021 08:42
@loan-laux
Copy link
Contributor Author

Not sure why Snyk keeps on failing. I've resolved every security warning that npm audit was giving me. @Akarshit do you know more about what's wrong here?

@Akarshit
Copy link
Contributor

Screen Shot 2021-08-12 at 12 27 35 PM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants