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

Docs: Adds missing snippets args page #23665

Merged
merged 2 commits into from
Aug 1, 2023
Merged

Conversation

jonniebigodes
Copy link
Contributor

With this pull request the args documentation is updated to fix a couple of missing snippets that were probably accidentally removed here.

Reported here

What I did

  • Reintroduced the missing snippets

@kylegach I've assigned this to you for you to take a look at it. Add any labels and feedback you deem fit to make the snippets available. Appreciate it 🙏

How to test

Checklist

  • Make sure your changes are tested (stories and/or unit, integration, or end-to-end tests)
  • Make sure to add/update documentation regarding your changes
  • If you are deprecating/removing a feature, make sure to update
    MIGRATION.MD

Maintainers

  • When this PR is ready for testing, make sure to add ci:normal, ci:merged or ci:daily GH label to it to run a specific set of sandboxes. The particular set of sandboxes can be found in code/lib/cli/src/sandbox-templates.ts
  • Make sure this PR contains one of the labels below.

["cleanup", "BREAKING CHANGE", "feature request", "bug", "build", "documentation", "maintenance", "dependencies", "other"]

🦋 Canary release

This PR does not have a canary release associated. You can request a canary release of this pull request by mentioning the @storybookjs/core team here.

core team members can create a canary release here or locally with gh workflow run --repo storybookjs/storybook canary-release-pr.yml --field pr=<PR_NUMBER>

@jonniebigodes jonniebigodes added documentation ci:docs Run the CI jobs for documentation checks only. labels Jul 31, 2023
@jonniebigodes jonniebigodes requested a review from kylegach as a code owner July 31, 2023 13:40
- Rename snippets to match naming schema
- Fix incorrect framing for example
- Add disclaimer that globals is probably a better option
@kylegach kylegach force-pushed the chore_docs_fix_args_snippets branch from a87451d to 50a7d85 Compare August 1, 2023 16:49
@kylegach kylegach added the patch:yes Bugfix & documentation PR that need to be picked to main branch label Aug 1, 2023
@jonniebigodes
Copy link
Contributor Author

jonniebigodes commented Aug 1, 2023

@kylegach thanks for taking the time to tweak the text to add some clarification on usage and aligning the snippets. Appreciate it. Merging it now.

@jonniebigodes jonniebigodes merged commit be7e95f into next Aug 1, 2023
@jonniebigodes jonniebigodes deleted the chore_docs_fix_args_snippets branch August 1, 2023 18:31
storybook-bot pushed a commit that referenced this pull request Aug 1, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
@github-actions github-actions bot mentioned this pull request Aug 1, 2023
33 tasks
storybook-bot pushed a commit that referenced this pull request Aug 1, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
@github-actions github-actions bot mentioned this pull request Aug 1, 2023
14 tasks
storybook-bot pushed a commit that referenced this pull request Aug 1, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 2, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 2, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 2, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 2, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 3, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
@github-actions github-actions bot mentioned this pull request Aug 3, 2023
32 tasks
storybook-bot pushed a commit that referenced this pull request Aug 4, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 7, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 7, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 7, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 7, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 8, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 9, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 9, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
storybook-bot pushed a commit that referenced this pull request Aug 9, 2023
Docs: Adds missing snippets args page
(cherry picked from commit be7e95f)
@kasperpeulen kasperpeulen removed the patch:yes Bugfix & documentation PR that need to be picked to main branch label Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:docs Run the CI jobs for documentation checks only. documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants