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

prop type height is invalid #12

Closed
fgvicente opened this issue Mar 14, 2018 · 4 comments
Closed

prop type height is invalid #12

fgvicente opened this issue Mar 14, 2018 · 4 comments

Comments

@fgvicente
Copy link

Warning: Failed prop type: PureChart: prop type height is invalid; it must be a function, usually from the prop-types package, but received number.

  • node_modules/fbjs/lib/warning.js:33:20 in printWarning
  • node_modules/fbjs/lib/warning.js:57:25 in warning
  • node_modules/prop-types/checkPropTypes.js:52:18 in checkPropTypes
  • node_modules/react/cjs/react.development.js:1171:19 in validatePropTypes
  • node_modules/react/cjs/react.development.js:1268:22 in createElementWithValidation
  • pages/Living.js:16:24 in render
  • node_modules/react-proxy/modules/createPrototypeProxy.js:44:35 in proxiedMethod
  • node_modules/react-native/Libraries/Renderer/ReactNativeRenderer-dev.js:7866:21 in finishClassComponent
  • ... 18 more stack frames from framework internals
@isthaison
Copy link

PureChart.propTypes = {
type: PropTypes.string.isRequired,
data: PropTypes.array.isRequired,
color: PropTypes.string,
-- height: PropTypes.number,
}
PureChart.defaultProps = {
color: '#297AB1',
--height:100
}

@oksktank
Copy link
Owner

Oh it was my mistake.

I fixed it and released 0.0.18.
Thank you for reporting.

@isthaison
Copy link

when is you release chart pie ?

@oksktank
Copy link
Owner

oksktank commented Mar 17, 2018

@isthaison we tried hard to implement pie chart, but we got some trouble in android because it doesn't support skew transform. we've found another way to implement pie chart both android and ios.
It will be released within three weeks.

Thank you

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

No branches or pull requests

3 participants