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

new merge #221

Open
wants to merge 70 commits into
base: 2018
Choose a base branch
from
Open

new merge #221

wants to merge 70 commits into from

Conversation

Lostinaa
Copy link

merge

wesbos and others added 30 commits February 28, 2018 12:38
A few suggested changes to make the meaning clearer.
Cleaned up language in the To Start section
updated firebase to 4.10.1
Updated package versions to fix vulnerability issues
using `this.myInput.value` gives undefined.
Change "current.value" to use new ref syntax
change all instances of "value.value"
I've seen quite a few people run into this issue in the Slack channel, myself included. You go over this in the Apache deploy video, but most people will try interacting with their app in the Netlify or Now deploy videos and think it's broken.
Added some missing prop-types
damsalem and others added 30 commits February 6, 2019 20:59
Updated ReadMe note covering dependencies
Update styles:watch command to use -- to prevent duplication
Removing incompatible  browerslist flag
Add sans-serif fallback to button and input styles
Add missing propTypes property (deleteFish).
Add missing propTypes property (deleteFish)
Update starter and finished app npm packages
add missing images route for vercel deployments
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.