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 to Meteor 2.0, React 16.14.0 and enable HMR #344

Merged
merged 8 commits into from
Feb 26, 2021

Conversation

loan-laux
Copy link
Contributor

@loan-laux loan-laux commented Feb 4, 2021

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. Also update to React 16.14.0 as HMR needs React > 16.9.0 to work and we were running 16.8.6.

Breaking changes

None.

Testing

  1. Start reaction-admin, 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.

Signed-off-by: Loan Laux <loan@outgrow.io>
Signed-off-by: Loan Laux <loan@outgrow.io>
Signed-off-by: Loan Laux <loan@outgrow.io>
Signed-off-by: Loan Laux <loan@outgrow.io>
@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.

Signed-off-by: Loan Laux <loan@outgrow.io>
…to loan-update-meteor-2.0

Signed-off-by: Loan Laux <loan@outgrow.io>
@Akarshit
Copy link
Contributor

@loan-laux Can you please update the PR with latest trunk, I tried doing that but I don't have write permission to your org.

@Akarshit Akarshit self-requested a review February 15, 2021 20:32
@loan-laux
Copy link
Contributor Author

@Akarshit Done. ✅

Signed-off-by: Loan Laux <loan@outgrow.io>
@Akarshit Akarshit merged commit 00ebb33 into reactioncommerce:trunk Feb 26, 2021
@Akarshit Akarshit mentioned this pull request Mar 3, 2021
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

2 participants