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

Update to be compatible with React 15.4 #82

Merged
merged 1 commit into from
Nov 16, 2016

Conversation

thisbejim
Copy link

As per facebook#7770 some dependencies in React 15.4 were moved from react/lib/ to react-dom/lib/.

@antoinerousseau
Copy link

antoinerousseau commented Nov 16, 2016

Can someone please merge this PR? Now that React 15.4 is published on NPM...

cc @madjam002

@shai32
Copy link

shai32 commented Nov 16, 2016

+1

2 similar comments
@samrispaud
Copy link

+1

@krazik
Copy link

krazik commented Nov 16, 2016

+1

@krazik
Copy link

krazik commented Nov 16, 2016

@zilverline-tech

@jbblanchet
Copy link

I don't want to be rude or anything, but I'm not sure all the +1 are really that helpful. If you want to voice your support, it is possible to add your reaction to the initial post. This way, people subscribing to the issue won't be spammed for nothing. 😅

@kyteague
Copy link

react-dom should get added to the peer dependencies as well in package.json

@s0meone
Copy link

s0meone commented Nov 16, 2016

Yes. We've heard you. Spamming the thread with +1 has absolutely no effect other than annoying people.

Replacing the imports didn't fix the plugin. I found the issue and I'm currently preparing a new release.

@s0meone s0meone merged commit 8f6efa8 into zilverline:master Nov 16, 2016
@gaearon
Copy link

gaearon commented Nov 16, 2016

Thank you for quick response!

petetnt added a commit to petetnt/admin-on-rest that referenced this pull request Nov 17, 2016
... to fix React 15.4.0 compatibility). 

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin
petetnt added a commit to petetnt/admin-on-rest that referenced this pull request Nov 18, 2016
... to fix React 15.4.0 compatibility).

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin

Update React too for correct peer deps

Signed-off-by: petetnt <pete.a.nykanen@gmail.com>
fzaninotto pushed a commit to petetnt/admin-on-rest that referenced this pull request Nov 30, 2016
... to fix React 15.4.0 compatibility).

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin

Update React too for correct peer deps

Signed-off-by: petetnt <pete.a.nykanen@gmail.com>
Granddevv added a commit to Granddevv/React-Admin-panel-restful-apis- that referenced this pull request Jan 15, 2018
... to fix React 15.4.0 compatibility). 

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin
Granddevv added a commit to Granddevv/React-Admin-panel-restful-apis- that referenced this pull request Jan 15, 2018
... to fix React 15.4.0 compatibility).

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin

Update React too for correct peer deps

Signed-off-by: petetnt <pete.a.nykanen@gmail.com>
coderbag added a commit to coderbag/admin-on-rest that referenced this pull request Jan 20, 2018
... to fix React 15.4.0 compatibility). 

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin
coderbag added a commit to coderbag/admin-on-rest that referenced this pull request Jan 20, 2018
... to fix React 15.4.0 compatibility).

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin

Update React too for correct peer deps

Signed-off-by: petetnt <pete.a.nykanen@gmail.com>
smartgru pushed a commit to smartgru/react-admin-panel that referenced this pull request Dec 5, 2018
... to fix React 15.4.0 compatibility). 

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin
smartgru pushed a commit to smartgru/react-admin-panel that referenced this pull request Dec 5, 2018
... to fix React 15.4.0 compatibility).

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin

Update React too for correct peer deps

Signed-off-by: petetnt <pete.a.nykanen@gmail.com>
smartgru added a commit to smartgru/react-admin-panel that referenced this pull request Aug 14, 2021
... to fix React 15.4.0 compatibility). 

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin
smartgru added a commit to smartgru/react-admin-panel that referenced this pull request Aug 14, 2021
... to fix React 15.4.0 compatibility).

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin

Update React too for correct peer deps

Signed-off-by: petetnt <pete.a.nykanen@gmail.com>
DarkNight918 added a commit to DarkNight918/admin-on-rest that referenced this pull request Mar 21, 2023
... to fix React 15.4.0 compatibility). 

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin
DarkNight918 added a commit to DarkNight918/admin-on-rest that referenced this pull request Mar 21, 2023
... to fix React 15.4.0 compatibility).

[react-v15.4.0](https://facebook.github.io/react/blog/2016/11/16/react-v15.4.0.html) broke `react-tap-event-plugin@v1`, which got updated to 2.0.1. just now at zilverline/react-tap-event-plugin#82. See the change set here: https://github.com/zilverline/react-tap-event-plugin/pull/82/files. It only changes the paths to `react-dom`.

https://www.npmjs.com/package/react-tap-event-plugin

Update React too for correct peer deps

Signed-off-by: petetnt <pete.a.nykanen@gmail.com>
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.

9 participants