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

Change to ReactDOM.findDOMNode() and use separated react addon libs #1733

Closed
wants to merge 5 commits into from
Closed

Conversation

Aaike
Copy link
Contributor

@Aaike Aaike commented Sep 26, 2015

No description provided.

@@ -61,9 +65,7 @@
"mocha": "^2.2.5",
"phantomjs": "^1.9.17",
"react-hot-loader": "^1.2.8",
"react-router": "^0.13.3",
"react-stub-context": "^0.3.0",
Copy link
Member

Choose a reason for hiding this comment

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

travis complains: Error: Cannot find module 'react-stub-context'.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeh react-stub-context hasn't been updated yet to react 0.14.
@shaurya947 said to just leave it out if it isn't used by the files source files : #1704

Copy link
Member

Choose a reason for hiding this comment

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

Does this means that we can't fix the build until react-stub-context support react v0.14?
I will close #1725 when the build pass since I believe this PR is more appropriate.

Copy link
Contributor

Choose a reason for hiding this comment

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

@Aaike @oliviertassinari I suggested leaving it out because I thought that might fix the build error.

What can we do now to make the build successful?

Copy link
Member

Choose a reason for hiding this comment

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

@shaurya947 I'm gonna try to fix react-stub-context.

Copy link
Contributor

Choose a reason for hiding this comment

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

👍

This was referenced Sep 28, 2015
@oliviertassinari
Copy link
Member

I think that we can close this PR since #1751 was merged.

@shaurya947 shaurya947 closed this Sep 30, 2015
mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Nov 10, 2020
Bumps [babel-loader](https://github.com/babel/babel-loader) from 8.0.6 to 8.1.0.
- [Release notes](https://github.com/babel/babel-loader/releases)
- [Changelog](https://github.com/babel/babel-loader/blob/master/CHANGELOG.md)
- [Commits](babel/babel-loader@v8.0.6...v8.1.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
@zannager zannager added the docs Improvements or additions to the documentation label Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants