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

[UiActions] fix race condition registering actions #90944

Merged
merged 1 commit into from
Feb 11, 2021

Conversation

Dosant
Copy link
Contributor

@Dosant Dosant commented Feb 10, 2021

Summary

fixes #90929

Moves attaching value_click and range_select actions in data plugin from setup to start hook because of race condition that trigger might be not yet registered by embeddable plugin.

Testing

Worth testing anything you can think of related to click/range_select interactions on a dashboard and inside visualization editors which is not covered by functional tests. Drilldowns included. Pay attention to the order of actions in a context menu.

@Dosant Dosant added release_note:skip Skip the PR/issue when compiling release notes Team:AppServices v7.12.0 v8.0.0 Feature:UIActions UI actions. These are client side only, not related to the server side actions.. labels Feb 10, 2021
@Dosant Dosant marked this pull request as ready for review February 10, 2021 15:50
@Dosant Dosant requested a review from a team as a code owner February 10, 2021 15:50
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-services (Team:AppServices)

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

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

id before after diff
data 797.9KB 797.8KB -48.0B

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

Copy link
Contributor

@streamich streamich left a comment

Choose a reason for hiding this comment

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

Code LGTM, I'll test run it on my PC in a couple of hours.

Copy link
Contributor

@streamich streamich left a comment

Choose a reason for hiding this comment

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

Checked on my laptop, all seems to be working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:UIActions UI actions. These are client side only, not related to the server side actions.. release_note:skip Skip the PR/issue when compiling release notes v7.12.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UIActions] Embeddable and data plugin triggers/actions race condition
4 participants