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] [Vega] Better error explanation for EsErrors and inspector now showing error responses (#112634) #112932

Merged
merged 1 commit into from
Sep 23, 2021

Conversation

kibanamachine
Copy link
Contributor

Backports the following commits to 7.x:

…g error responses (elastic#112634)

* 🐛 Fix EsError not showing info

* 🐛 Record error response in the inspector

* 🏷️ Fix type issue

* 👌 Integrate feedback

* 👌 Integrated latest feedback

* 👌 i18n unknwon message

* 👌 Integrate feedback

* 🐛 Fix syntax error

* 🏷️ Fix type issue

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine
Copy link
Contributor Author

💚 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 781.6KB 782.0KB +414.0B
visTypeVega 55.6KB 55.6KB +99.0B
total +513.0B

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

cc @dej611

@kibanamachine kibanamachine merged commit ea44eb7 into elastic:7.x Sep 23, 2021
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