Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix with-amp example #3153

Merged
merged 1 commit into from
Oct 23, 2017
Merged

Fix with-amp example #3153

merged 1 commit into from
Oct 23, 2017

Conversation

ismamz
Copy link
Contributor

@ismamz ismamz commented Oct 23, 2017

React v16 supports invalid DOM attributes now.
ReactInjection and DOMProperty throws an error.
Reference: #3152

React v16 supports invalid DOM attributes now. 
ReactInjection and DOMProperty throws an error.
Reference: #3152
Copy link
Member

@timneutkens timneutkens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@timneutkens timneutkens merged commit 8c60a2e into vercel:master Oct 23, 2017
@ismamz
Copy link
Contributor Author

ismamz commented Oct 23, 2017

Hi @timneutkens! There are some examples with errors, some of them throw a compatibility error between React < v16 and Next v4, and there are issues like this one, with unknown DOM attributes.

Do you think that all examples should update to next@latest and react@16.0.0?
I can help if you need.

I see that you updated many examples in #3113, but some were not updated.

@timneutkens
Copy link
Member

If you could check them out that'd be great, I just did a simple search/replace

@lock
Copy link

lock bot commented May 10, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked as resolved and limited conversation to collaborators May 10, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants