Skip to content

Conversation

@sophiebits
Copy link
Collaborator

Apparently we did a lot.

To curious onlookers: Please no tweeting about this yet; we're not quite done. :) We'll try to cut the release soon.

Preview:

@gaearon
Copy link
Collaborator

gaearon commented Sep 4, 2015

Amazing job @spicyj @cpojer @zpao @sebmarkbage & co, this is going to be my favorite release so far!

@sophiebits sophiebits mentioned this pull request Sep 4, 2015
@mridgway
Copy link
Contributor

mridgway commented Sep 5, 2015

The deprecation of this.getDOMNode() should probably refer to ReactDOM.getDOMNode(this) instead of React.getDOMNode(this) since it is deprecated in React package.

Can't wait for this! 👍

@sophiebits
Copy link
Collaborator Author

Thank you, good catch.

@sophiebits sophiebits force-pushed the rc14blog branch 2 times, most recently from 754fed7 to 1336f1e Compare September 5, 2015 00:50
@knowbody
Copy link

knowbody commented Sep 5, 2015

Great work! Huge release, looking forward to it

@korczis
Copy link

korczis commented Sep 5, 2015

👍

@alexeyraspopov
Copy link
Contributor

How stateless function components supposed to work? Or should I still use classes to wrap these stateless functions?

@bloodyowl
Copy link
Contributor

great job. can't wait for this release!

Copy link
Contributor

Choose a reason for hiding this comment

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

@spicyj react-addons-perf and react-addons-test-utils too?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, thanks.

@quantizor
Copy link
Contributor

@alexeyraspopov It was explained pretty clearly in the post, IMO. Instead of having to createClass or extend React.Component, you simply feed a function with props as the primary argument and it takes the static output from it to render.

@alexeyraspopov
Copy link
Contributor

@yaycmyk I've tried to do it with 0.14-beta3 and it doesn't work.

@quantizor
Copy link
Contributor

@alexeyraspopov AFAIK it's not actually merged yet - #3995

@sophiebits
Copy link
Collaborator Author

@alexeyraspopov It's not in beta3. Please wait for the RC.

@alexeyraspopov
Copy link
Contributor

@spicyj but what about #4587? Should it work now?

@sophiebits sophiebits added this to the 0.14 milestone Sep 9, 2015
@sophiebits
Copy link
Collaborator Author

@alexeyraspopov That landed after beta3 was cut. It'll be in the RC that goes out sometime soon.

Copy link
Member

Choose a reason for hiding this comment

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

What's with indenting paragraphs?

Copy link
Member

Choose a reason for hiding this comment

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

s/it's/it has/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Makes it part of the same bullet point.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ack

Copy link
Member

Choose a reason for hiding this comment

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

Oh that's a giant list… that was hard to see in the diff. I guess that's ok (and then you can ignore the h4 comment).

@sophiebits
Copy link
Collaborator Author

(marking accepted per 1:1 chat)

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.