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

[Legacy Controls] Decouple DEPRECATION_BADGE action from Embeddable framework #175928

Merged

Conversation

Heenawter
Copy link
Contributor

@Heenawter Heenawter commented Jan 30, 2024

Part of #175138

Summary

This PR decouples the DEPRECATION_BADGE action (which is used specifically for the legacy input controls panel) from Embeddable framework by migrating to sets of composable interfaces.

Testing:
Since the legacy input control vis is deprecated and doesn't show up in the "Add panel" menu anymore, you need to navigate to <base url>/app/visualize#/create?type=input_control_vis in order to create this panel type for testing.

For maintainers

@Heenawter Heenawter added Feature:Embedding Embedding content via iFrame Feature:Input Control Input controls visualization Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas loe:small Small Level of Effort release_note:skip Skip the PR/issue when compiling release notes impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Feature:Embeddables Relating to the Embeddable system labels Jan 30, 2024
@Heenawter Heenawter self-assigned this Jan 30, 2024
@Heenawter
Copy link
Contributor Author

/ci

@Heenawter
Copy link
Contributor Author

/ci

@Heenawter Heenawter changed the title [Legacy Controls] Decouple DEPRECATION_BADGE action from Embeddable framework [Legacy Controls] Decouple DEPRECATION_BADGE action from Embeddable framework Jan 30, 2024
@Heenawter Heenawter marked this pull request as ready for review January 30, 2024 21:52
@Heenawter Heenawter requested a review from a team as a code owner January 30, 2024 21:52
@elasticmachine
Copy link
Contributor

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

@nreese nreese self-requested a review January 30, 2024 23:52
Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

LGTM
code review and tested in chrome

src/plugins/input_control_vis/public/deprecation_badge.ts Outdated Show resolved Hide resolved
@Heenawter Heenawter enabled auto-merge (squash) January 31, 2024 16:11
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
inputControlVis 46 66 +20

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
inputControlVis 6.7KB 7.3KB +592.0B

History

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

cc @Heenawter

@Heenawter Heenawter merged commit b6fbae2 into elastic:main Jan 31, 2024
17 checks passed
@kibanamachine kibanamachine added v8.13.0 backport:skip This commit does not require backporting labels Jan 31, 2024
@Heenawter Heenawter deleted the migrate-deprecation-badge-action_2024-01-30 branch January 31, 2024 17:43
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this pull request Feb 15, 2024
… framework (elastic#175928)

Part of elastic#175138

## Summary

This PR decouples the `DEPRECATION_BADGE` action (which is used
specifically for the legacy input controls panel) from Embeddable
framework by migrating to sets of composable interfaces.

**Testing:**
Since the legacy input control vis is deprecated and doesn't show up in
the "Add panel" menu anymore, you need to navigate to `<base
url>/app/visualize#/create?type=input_control_vis` in order to create
this panel type for testing.

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
fkanout pushed a commit to fkanout/kibana that referenced this pull request Mar 4, 2024
… framework (elastic#175928)

Part of elastic#175138

## Summary

This PR decouples the `DEPRECATION_BADGE` action (which is used
specifically for the legacy input controls panel) from Embeddable
framework by migrating to sets of composable interfaces.

**Testing:**
Since the legacy input control vis is deprecated and doesn't show up in
the "Add panel" menu anymore, you need to navigate to `<base
url>/app/visualize#/create?type=input_control_vis` in order to create
this panel type for testing.

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting Feature:Embeddables Relating to the Embeddable system Feature:Embedding Embedding content via iFrame Feature:Input Control Input controls visualization impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. loe:small Small Level of Effort project:embeddableRebuild release_note:skip Skip the PR/issue when compiling release notes Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants