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

[PoC] Storybook for package scss - vanilla #1277

Closed
wants to merge 8 commits into from

Conversation

lucienbertin
Copy link
Contributor

@lucienbertin lucienbertin commented Apr 15, 2021

adds a storybook sandbox at the root of lf repo

doesn't use a js framework (vue, angular, react) for more easier usage


EDIT won't scale - don't merge - see #1283

this PoC was with the approach of one storybook at LF root for all 3 packages . one storybook to rule them all, and in darkness bind them.

But but but that wont work cuz until SBv7 there is no support for multiple frameworks, and the only official solution for mono repos with multiple frameworks is composaition (github issue, sb doc) and the way it works is you have N micro SB portals that you feed into 1 macro SB portal.

so for it to work we will need 1 SB portal for each package (icons, scss, ng)

@lucienbertin lucienbertin added enhancement 🔨 Technical Doesn't affect the output (refactor, dependencies update, cleaning, etc.) labels Apr 15, 2021
@lucienbertin lucienbertin requested review from a team as code owners April 15, 2021 08:45
@LuccaIntegration
Copy link

jenkins auto deploy http://lucca-front.lucca.local/PR-1277

@lucienbertin lucienbertin self-assigned this Apr 15, 2021
@LuccaIntegration
Copy link

jenkins auto deploy http://lucca-front.lucca.local/PR-1277

@LuccaIntegration
Copy link

jenkins auto deploy http://lucca-front.lucca.local/PR-1277

@lucienbertin
Copy link
Contributor Author

not the implementation chosen, see #1286

@lucienbertin lucienbertin deleted the poc.storybook.html branch May 5, 2021 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Technical Doesn't affect the output (refactor, dependencies update, cleaning, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants