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

[7.x] [Actions] Adding hasAuth to Webhook Configuration to avoid confusing UX (#81778) #81823

Merged
merged 3 commits into from
Oct 28, 2020

Conversation

ymao1
Copy link
Contributor

@ymao1 ymao1 commented Oct 27, 2020

Backports the following commits to 7.x:

…g UX (elastic#81778)

* Adding hasAuth to server and client

* Adding migration and fixing tests

* Fixing test

* Adding spacing

* Adding functional test

* Fixing migration

Co-authored-by: Kibana Machine <[email protected]>
@ymao1 ymao1 added the backport label Oct 27, 2020
@ymao1
Copy link
Contributor Author

ymao1 commented Oct 27, 2020

@elasticmachine merge upstream

@ymao1
Copy link
Contributor Author

ymao1 commented Oct 27, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

async chunks size

id before after diff
triggersActionsUi 1.5MB 1.5MB +1.0KB

page load bundle size

id before after diff
triggersActionsUi 144.5KB 145.0KB +596.0B

History

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

@ymao1 ymao1 merged commit 9ed004c into elastic:7.x Oct 28, 2020
@ymao1 ymao1 deleted the backport/7.x/pr-81778 branch February 4, 2021 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants