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

📕 Storybook 3 #55248

Closed
wants to merge 7 commits into from
Closed

📕 Storybook 3 #55248

wants to merge 7 commits into from

Conversation

streamich
Copy link
Contributor

@streamich streamich commented Jan 17, 2020

Summary

  • Upgrades Storybook to 5.3.x.
  • Switches to @kbn/ui-shared-deps instead of custom DLL.
  • Adds a sample story to embeddable plugin that uses Storybook knobs.
  • Partially addresses 📕 Storybook Roadmap #54707

@streamich streamich requested review from a team as code owners January 17, 2020 22:33
@streamich streamich requested review from spalger and smith January 17, 2020 22:33
@streamich streamich added release_note:skip Skip the PR/issue when compiling release notes review Feature:Storybook v7.7.0 v8.0.0 Team:AppArch Team:Operations Team label for Operations Team labels Jan 17, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-arch (Team:AppArch)

@streamich streamich changed the title Storybook 3 📕 Storybook 3 Jan 17, 2020
@streamich streamich mentioned this pull request Jan 17, 2020
11 tasks
@kibanamachine
Copy link
Contributor

💔 Build Failed

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Copy link
Contributor

@smith smith left a comment

Choose a reason for hiding this comment

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

Running yarn storybook apm with this gives me a very long output of errors that looks like:

.js
              Field 'browser' doesn't contain a valid alias configuration
              /Users/smith/Code/kibana/node_modules/react-syntax-highlighter/prism-light.js doesn't exist

over and over for every location it's trying.

The actual file it's looking for is in node_modules/react-syntax-highlighter/dist/cjs/prism-light.js or node_modules/react-syntax-highlighter/dist/esm/prism-light.js.

The package.json for that module (which is a direct dependency in x-pack's package.json and used by APM) looks normal and defines main and module keys for the index.js files in those respective cjs and esm directories.

I couldn't think of what to do to the kbn-storybook webpack config to fix this and googling wasn't helpful for me.

Please try it and make sure it happens for you too. I did a yarn kbn bootstrap before trying.

@spalger spalger added v7.7.1 and removed v7.7.0 labels May 14, 2020
@spalger spalger marked this pull request as draft May 28, 2020 16:06
@spalger spalger removed their request for review May 28, 2020 16:06
@streamich streamich closed this Jun 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Storybook release_note:skip Skip the PR/issue when compiling release notes review Team:Operations Team label for Operations Team v7.7.1 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants