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

Console Error Running Storybook #2

Open
mcastre opened this issue Aug 12, 2020 · 4 comments · May be fixed by #4
Open

Console Error Running Storybook #2

mcastre opened this issue Aug 12, 2020 · 4 comments · May be fixed by #4

Comments

@mcastre
Copy link

mcastre commented Aug 12, 2020

Hello @audriusnavickasDB! I read your article and pulled down your repo to try it out, but after npm installing and npm start, the app loads a blank screen, most likely due to this console error:

Warning: Failed prop type: The prop `style.container` is marked as required in `ContainerDecorator`, but its value is `undefined`.
    in ContainerDecorator (created by Context.Consumer)
    in Unknown (created by NodeHeader)
    in NodeHeader (created by TreeNode)
    in li (created by Context.Consumer)
    in Styled(li) (created by TreeNode)
    in TreeNode (created by TreeBeard)
    in ul (created by Context.Consumer)
    in Styled(ul) (created by TreeBeard)
    in TreeBeard (created by Stories)
    in div (created by Context.Consumer)
    in Styled(div) (created by ForwardRef)
    in ForwardRef (created by Stories)
    in Stories (created by StoriesPanel)
    in div (created by Context.Consumer)
    in Styled(div) (created by ForwardRef)
    in ForwardRef (created by StoriesPanel)
    in StoriesPanel (created by Container(StoriesPanel))
    in Container(StoriesPanel) (created by storiesPanel)
    in storiesPanel (created by Layout)
    in div (created by Context.Consumer)
    in Styled(div) (created by ForwardRef)
    in ForwardRef (created by Layout)
    in div (created by Context.Consumer)
    in Styled(div) (created by ForwardRef)
    in ForwardRef (created by Layout)
    in div (created by Pane)
    in Pane (created by SplitPane)
    in div (created by SplitPane)
    in SplitPane (created by Layout)
    in Layout (created by Layout)
    in div (created by Context.Consumer)
    in Styled(div) (created by ForwardRef)
    in ForwardRef (created by Layout)
    in div (created by Layout)
    in Unknown (created by Layout)
    in Layout (created by Container(Layout))
    in Container(Layout)
    in div

i haven't found anything in the repo regarding this error so I assume its an issue with Storybook generating the components. i also tried building the components with lerna first, but that did not resolve the issue.

@ananddharne
Copy link

did you find a resolution? @mcastre same error for me

@MichaelDimmitt
Copy link

MichaelDimmitt commented May 23, 2021

probably an issue because there is no package-lock.json

I wonder if it is related to the version of node or version of npm

  1. going to try a few different npm versions.
  2. also looking at upgrading storybook is an idea.

I am guessing that it was working when uploaded and the versions are hardcoded into the package.json

running yarn install I got the following error:

error [email protected]: The engine "node" is incompatible with this module. Expected version ">=10". Got "8.17.0"
error Found incompatible module.

@MichaelDimmitt
Copy link

MichaelDimmitt commented May 23, 2021

It is fixed in pr #1

see the comment I left in there
#1 (comment)

A few things updated in that pr!

Storybook was updated!
also node-sass version for master is currently pointed to 4.11.0 which is no longer a valid release on node-sass 🤦‍♂️
so when you try to install, it will fail if you do not already have it on your computer.

The person who made that pr upgraded to ^4.13.0

@ananddharne @mcastre

@MichaelDimmitt MichaelDimmitt linked a pull request May 23, 2021 that will close this issue
@MichaelDimmitt
Copy link

So the author was definately on node version less than 12 😅

Screen Shot 2021-05-23 at 4 27 10 PM

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 a pull request may close this issue.

3 participants