Skip to content

Fix blog post content about DOM Attributes #126

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

Merged
merged 2 commits into from
Oct 11, 2017
Merged

Conversation

nikgraf
Copy link
Contributor

@nikgraf nikgraf commented Oct 10, 2017

As discussed with Dan here the blog post is not correct and should be fix. Let me know if I should change the wording though.

@reactjs-bot
Copy link

reactjs-bot commented Oct 10, 2017

Deploy preview ready!

Built with commit 8c04bfc

https://deploy-preview-126--reactjs.netlify.com

@@ -163,7 +163,7 @@ Below is a detailed list of them.
```

React 15: Converts `NaN`s to strings and passes them through.
React 16: Warns and ignores them.
React 16: Converts `NaN`s to strings and warns about them.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we clarify a bit more?

React 16: Converts NaNs to strings and passes them through with a warning.

@nikgraf
Copy link
Contributor Author

nikgraf commented Oct 11, 2017

@bvaughn sounds good, just updated the PR

Copy link
Contributor

@bvaughn bvaughn left a comment

Choose a reason for hiding this comment

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

Thank you~

@bvaughn bvaughn merged commit 29cd12a into reactjs:master Oct 11, 2017
@nikgraf nikgraf deleted the patch-1 branch October 11, 2017 18:12
jhonmike pushed a commit to jhonmike/reactjs.org that referenced this pull request Jul 1, 2020
* Translate getting started

* Apply translation review suggestions

* Apply translation review suggestions

* Fix something missing title
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants