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

Angular: Add global CSF3 renderer #15742

Merged
merged 5 commits into from Aug 7, 2021

Conversation

yngvebn
Copy link
Contributor

@yngvebn yngvebn commented Aug 3, 2021

Issue: #15498

What I did

Added a basic global renderer for Angular and CSF3
Enabled previewCsfV3 for the angular-cli example storybook

How to test

  • Is this testable with Jest or Chromatic screenshots? Yes
  • Does this need a new example in the kitchen sink apps?
  • Does this need an update to the documentation?

If your answer is yes to any of these, please make sure to include it in your PR.

Added a couple of stories to examples/angular-cli

@nx-cloud
Copy link

nx-cloud bot commented Aug 3, 2021

Nx Cloud Report

CI ran the following commands for commit 37bbe90. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

Status Command
#000000 nx run-many --target=prepare --all --parallel --max-parallel=15

Sent with 💌 from NxCloud.

@yngvebn yngvebn marked this pull request as draft August 3, 2021 13:03
@yngvebn
Copy link
Contributor Author

yngvebn commented Aug 3, 2021

@shilman / @ThibaudAV I'm struggling with getting the tests to run green locally, even though I haven't touched any of the files or tests it's complaining about... Would greatly appreciate if someone would help out.. The same goes for this PR: #15743

import { InputComponent } from './sb-input.component';

export default {
title: 'CSF3/Input',
Copy link
Contributor

@ThibaudAV ThibaudAV Aug 3, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe Core / CSF3 / Input ? Or Beta / ... 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would "Preview" be a fitting prefix perhaps?

Copy link
Contributor

@ThibaudAV ThibaudAV Aug 5, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • In core I tried to place the stories showing and testing the native functionality of sotrybook
  • basics it's more for examples of storybook use or helpers or common examples
  • addons for addons
    ..
    so if CSF3 is to become something native in storybook main set, I can see it in core/...

but it's up to you, knowing that ;)

@yngvebn yngvebn marked this pull request as ready for review August 3, 2021 17:33
@yngvebn yngvebn requested a review from ThibaudAV August 3, 2021 17:33
@yngvebn yngvebn changed the title add global CSF3 renderer for Angular Angular: add global CSF3 renderer Aug 4, 2021
@shilman shilman changed the title Angular: add global CSF3 renderer Angular: Add global CSF3 renderer Aug 5, 2021
Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thanks @yngvebn 🚀

@shilman shilman added this to the 6.4 PRs milestone Aug 7, 2021
@yngvebn
Copy link
Contributor Author

yngvebn commented Aug 7, 2021

Awesome! Thanks @yngvebn 🚀

You bet!

@shilman shilman merged commit 17503b3 into storybookjs:next Aug 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants