React16 API

[ScriptedAlchemy]

on npm as @react16

This is a breaking change

[jaybe78]

@ScriptedAlchemy When do you plan to release this ?

[ScriptedAlchemy]

You tell me it’s stable. And I’ll release a flag tonight on NPM for you. Then roll it as a major release in a few days

[hedgepigdaniel]

Nice, this seems easier to follow than before.

Since getDerivedStateFromProps is only available in React 16.3, the peerDependency on react should probably be changed from * to ^16.3.0.

[jaybe78]

@ScriptedAlchemy I'm using it in my project. It is stable

[ScriptedAlchemy]

@jaybe78 anything you'd like to PR into it, take a look at the branch and then I'll give it a final brush, merge, and major release.

@GuillaumeCisco is there any way we can collab on this and make it one major release for both redux 6 and react 17?

[jaybe78]

@ScriptedAlchemy Would be nice to upgrade to babel 7 as well as part of that change. I did it in my branch, it's a seamless change: https://github.com/jaybe78/react-universal-component/blob/master/package.json
Other than that il all looks good and ready to be merged.
Thanks

[GuillaumeCisco]

@ScriptedAlchemy, thanks for poking me towards this issue.
As I explained in #172, we should only document the way users should work with [email protected] as the redux store will not be available the same way.

[jaybe78]

@ScriptedAlchemy Hello, any progress on this one ?

[ScriptedAlchemy]

Okay, i can merge this, Do you want to open a PR for babel 7? I dont mind moving to that and have done so in some places, this will be a major release

[ScriptedAlchemy]

@jaybe78 thanks for bumping this, ill merge it tomorrow. If i dont, bump it again - hold me to it. Squeaky wheel gets the grease and i oversee many projects

[jaybe78]

@ScriptedAlchemy hehe don't worry I will be a pain in the ass until you merge it. I will create PR for babel 7. Thanks :)

[ScriptedAlchemy]

Got some build errors I’m addressing, should merge it tomorrow /:

[ScriptedAlchemy]

PR opened here. some additional changes need to go in
#176

[ScriptedAlchemy]

[email protected]

[ScriptedAlchemy]

@jaybe78 One last time, will you try the released alpha I put up yesterday

Fixed the build issues so I'm ready to merge

[jaybe78]

@ScriptedAlchemy Will check it today, I'll get back to you ASAP. Thanks!

[ScriptedAlchemy]

Thanks I’ll merge if ur happy

[ScriptedAlchemy]

Merged via another PR which had other various updates