Skip to content

[canvas] Refactor Storybook from bespoke to standard configuration#101962

Merged
clintandrewhall merged 6 commits intoelastic:masterfrom
clintandrewhall:canvas/storybook_refactor
Jun 17, 2021
Merged

[canvas] Refactor Storybook from bespoke to standard configuration#101962
clintandrewhall merged 6 commits intoelastic:masterfrom
clintandrewhall:canvas/storybook_refactor

Conversation

@clintandrewhall
Copy link
Contributor

@clintandrewhall clintandrewhall commented Jun 10, 2021

Canvas was one of the first solutions to adopt Storybook, and had a lot of customizations that weren't quite supported by @kbn/storybook. With the introduction of kbn-shared-util and the growth of kbn-storybook-- as well as some welcome features like EUI theme switching-- I've been compelled to find the time to refactor our approach.

The result is awesome: a significant cut in code, faster load time, less complexity and more features.

@clintandrewhall clintandrewhall force-pushed the canvas/storybook_refactor branch from 6224928 to 261cb25 Compare June 11, 2021 01:16
@clintandrewhall clintandrewhall added Feature:Canvas impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:small Small Level of Effort review Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v7.14.0 v8.0.0 release_note:skip Skip the PR/issue when compiling release notes labels Jun 11, 2021
@clintandrewhall clintandrewhall marked this pull request as ready for review June 11, 2021 01:21
@clintandrewhall clintandrewhall requested a review from a team as a code owner June 11, 2021 01:21
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@clintandrewhall clintandrewhall requested review from smith and spalger June 11, 2021 01:22
@clintandrewhall clintandrewhall enabled auto-merge (squash) June 11, 2021 15:09
@clintandrewhall
Copy link
Contributor Author

@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.

tenor-33621576

@clintandrewhall
Copy link
Contributor Author

@elasticmachine merge upstream

@clintandrewhall
Copy link
Contributor Author

@elasticmachine merge upstream

@clintandrewhall
Copy link
Contributor Author

@elasticmachine merge upstream

@clintandrewhall clintandrewhall added the auto-backport Deprecated - use backport:version if exact versions are needed label Jun 17, 2021
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Jun 18, 2021
…101962) (#102589)

Co-authored-by: Clint Andrew Hall <clint.hall@elastic.co>
jloleysens added a commit to jloleysens/kibana that referenced this pull request Jun 18, 2021
…ets-tab

* 'master' of github.com:elastic/kibana: (93 commits)
  [ML] Remove blank job definition as it is unused and out-of-sync with Elasticsearch (elastic#102506)
  [Lens] Fix wrong error detection on transition to Top values operation (elastic#102384)
  [ML] Anomaly detection job custom_settings improvements (elastic#102099)
  [Cases] Route: Get all alerts attach to a case (elastic#101878)
  Fixes wrong list exception type when creating endpoint event filters list (elastic#102522)
  remove search bar that's not working yet (elastic#102550)
  Migrated Ingest Node Pipeline Functional Tests to use test_user (elastic#102409)
  [Maps] clean up feature editing name space to avoid conflicts with layer settings editing (elastic#102516)
  [canvas] Refactor Storybook from bespoke to standard configuration (elastic#101962)
  [Security Solution] adds wrapSequences method (RAC) (elastic#102106)
  [FTR] Stabilize SSLP functional tests (elastic#102553)
  [K8] Added `Inter` font files for new theme (elastic#102359)
  [Workplace Search] Convert Groups pages to new page template (elastic#102449)
  [DOC] Add experimental disclaimer to rollup jobs (elastic#95624)
  [Security Solution][Endpoint] Suppress some of the jest console.error noise created by endpoint list middelware (elastic#102535)
  [Fleet] Improve performance of Fleet setup (elastic#102219)
  [Alerting] Add event log entry when a rule starts executing (elastic#102001)
  [Fleet] Update docker image of registry used in integration tests (elastic#101911)
  [Asset Management] Osquery telemetry updates (elastic#100754)
  Converts saved object tagging to new management layout (elastic#102284)
  ...

# Conflicts:
#	x-pack/plugins/fleet/kibana.json
@clintandrewhall clintandrewhall deleted the canvas/storybook_refactor branch July 9, 2021 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Deprecated - use backport:version if exact versions are needed Feature:Canvas impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:small Small Level of Effort release_note:skip Skip the PR/issue when compiling release notes review Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v7.14.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants