Skip to content

Fix deprecation warnings for react >= 15.5.0 - #58

Merged
shepherdwind merged 3 commits into
react-component:masterfrom
y-yagi:migrate_to_react_v15.5.0
Apr 11, 2017
Merged

shepherdwind merged 3 commits into
react-component:masterfrom
y-yagi:migrate_to_react_v15.5.0

Conversation

@y-yagi

@y-yagi y-yagi commented Apr 10, 2017

Copy link
Copy Markdown
Contributor

Hi.

In React v15.5.0, React.PropTypes and React.createClass became deprecate.
So I migrated to prop-types package and create-react-class package.

@y-yagi y-yagi changed the title Migrate to react v15.5.0 Fix deprecation warnings for react >= 15.5.0 Apr 10, 2017
Comment thread src/Collapse.jsx Outdated
@@ -1,4 +1,5 @@
import React, { PropTypes, Children }from 'react';
import React, { Children }from 'react';

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

- import React, { Children }from 'react';
+ import React, { Children } from 'react';

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I fixed.

@afc163

afc163 commented Apr 10, 2017

Copy link
Copy Markdown
Member

Rebase master

@y-yagi
y-yagi force-pushed the migrate_to_react_v15.5.0 branch from f28ca8c to af485ce Compare April 10, 2017 03:11
@y-yagi
y-yagi force-pushed the migrate_to_react_v15.5.0 branch from af485ce to 75a3523 Compare April 10, 2017 03:14
@y-yagi

y-yagi commented Apr 10, 2017

Copy link
Copy Markdown
Contributor Author

Rebased with master.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.3%) to 96.377% when pulling 75a3523 on y-yagi:migrate_to_react_v15.5.0 into a4520b6 on react-component:master.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.3%) to 96.377% when pulling 75a3523 on y-yagi:migrate_to_react_v15.5.0 into a4520b6 on react-component:master.

Comment thread src/Collapse.jsx Outdated
}

const Collapse = React.createClass({
const createReactClass = require('create-react-class');

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Very thanks for you contribution.

But, I think It is better to change the createReactClass code to class style component.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you for your comment.
I changed the createReactClass code to class style component.
Please review.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.04%) to 97.159% when pulling 674ffc7 on y-yagi:migrate_to_react_v15.5.0 into a4520b6 on react-component:master.

@shepherdwind
shepherdwind merged commit 2c26e8e into react-component:master Apr 11, 2017
@y-yagi
y-yagi deleted the migrate_to_react_v15.5.0 branch April 11, 2017 01:54
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.

4 participants