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 package.json to CRA2 and cleanup React Component imports #6

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

Conversation

js-mode
Copy link

@js-mode js-mode commented Oct 18, 2018

Updated all of the package.json files to include proper exercise number in the name.

Updated all of the package.json files to include latest version of Create React App. Tested locally and yarn install works, but when one runs yarn install it will ask for the default browser to be added to the package.json automatically.

Also cleaned up the import React, { Component } from 'react'; statement so it is consistent throughout all of the exercises.

Added a few minor changes to make the event parameter more descriptive so it was event instead of e.

@js-mode
Copy link
Author

js-mode commented Oct 18, 2018

I just pulled down all of the latest changes from the master branch to make sure nothing was overwritten and the new App.solution.js file for Exercise 04 should be the latest version.

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.

1 participant