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

Bugfix story description first story #10171

Closed

Conversation

patelvp
Copy link

@patelvp patelvp commented Mar 20, 2020

Issue: #8093

What I did

  • expanded prop was set to false for Primary component which displays the first story. Hence the docs were not showing information as expected.
  • Setting expanded prop to true solves the issue.

How to test

  • Docs now appear for the first story.

AbdelhalimAhmed and others added 30 commits January 20, 2020 01:15
as I have an exception when using yarn with npx i forced it to use npm instead by: ```  --use-npm ```
Typescript: Export IStory in `@storybook/angular`
Fix broken link to repo in empty knobs panel
Addon-docs:  Fix TS false default value in prop table
L56 was missing an `=`, breaking `npm run storybook`
Addon-docs: Revert breaking source indentation fix
This may be intuitive enough to some, but it took me a minute to figure out so I think it'd be nice to have here.
Improve Typescript Configuration documentation page
shilman and others added 5 commits March 14, 2020 16:03
Unlike react (`src`), Ember has its application source files inside `app` directory. I think for beginner it is easy to understand if we stick to ember folder naming conventions.
@patelvp
Copy link
Author

patelvp commented Mar 20, 2020

This is my First open source contribution. Not an expert in opening a PR for forked repositories. Probably my fork was outdate. Hence all these commits. Can someone help me with this?

@shilman
Copy link
Member

shilman commented Mar 21, 2020

@patelvp please branch from next and target next in the PR. your code is branched from master.

@shilman shilman closed this Mar 21, 2020
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.