carteb / carte-blanche

An isolated development space with integrated fuzz testing for your components. See them individually, explore them in different states and quickly and confidently develop them.
https://www.youtube.com/watch?v=6g3-TQ6aaw8
MIT License
1.5k stars 47 forks source link

traverseParentPath(...) error when selecting a component #348

Closed bytewiz closed 8 years ago

bytewiz commented 8 years ago

Added one component to and then visited th carte-blanche but when selecting the component and the pop up shows this follow error is thrown:

Generating metadata failed Error: traverseParentPath(...): Cannot traverse from and to the same ID, ''.(…)(anonymous function) @ index.js:301

screen shot 2016-06-13 at 23 10 44
mxstbr commented 8 years ago

What does your component and layout of the component folder look like?

bytewiz commented 8 years ago

The App component which imports the List component

screen shot 2016-06-13 at 23 45 00

The List component.

screen shot 2016-06-13 at 23 43 01
bytewiz commented 8 years ago

@mxstbr any idea why this happens?

mxstbr commented 8 years ago

I think <div>this.props.firstName</div> is invalid JSX syntax, it should be <div>{ this.props.firstName }</div>, which might be the problem?

bytewiz commented 8 years ago

True my error 👍 but it still fails 😢 with same error @mxstbr

mxstbr commented 8 years ago

Hmm, not quite sure to be honest. Could you provide a contained repro case as a git repo? Then I'll take a look!

skayred commented 8 years ago

Can confirm the same behavour, see the sample project https://github.com/skayred/carte-blanche-hello-world

I can provide any additional info if needed

bytewiz commented 8 years ago

@skayred I can confirm the exact same behaviour in the sample project. same error.

mxstbr commented 8 years ago

Thanks for the repro case, will look into it!

ewnd9 commented 8 years ago

react@0.14 is the issue here, it's working with the latest react https://github.com/ewnd9/carte-blanche-hello-world

mxstbr commented 8 years ago

Hah, awesome, thanks for digging in @ewnd9, much appreciated! :+1:

ewnd9 commented 8 years ago

@mxstbr thanks for the project :clap: By the way, shouldn't be the react@15 requirement mentioned in the readme?

mxstbr commented 8 years ago

By the way, shouldn't be the react@15 requirement mentioned in the readme?

That sounds like a very reasonable idea, would you mind submitting a PR adding that? :pray:

ewnd9 commented 8 years ago

@mxstbr done https://github.com/carteb/carte-blanche/pull/356