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

Fatal error when closing a notification that has already cleared #19263

Closed
liza-mae opened this issue May 21, 2018 · 2 comments
Closed

Fatal error when closing a notification that has already cleared #19263

liza-mae opened this issue May 21, 2018 · 2 comments
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. v6.3.0

Comments

@liza-mae
Copy link
Contributor

Kibana version: 6.3.0 BC6
Elasticsearch version: 6.3.0 BC6
Server OS version: OSX
Browser version: Chrome / Firefox
Browser OS version: Latest
Original install method (e.g. download page, yum, from source, etc.): Staging

Describe the bug:
I noticed when closing a notification if you accidentally double click the 'X' or if the system is slow when you click to close the notification and it processes the action after it has cleared you get a fatal error. Seems like something we should handle more gracefully.

Error: Uncaught TypeError: Cannot read property 'clear' of undefined (https://localhost:5605/bundles/vendors.bundle.js:149)
at window.onerror (commons.bundle.js:3)
fatalError @ commons.bundle.js:3
window.onerror @ commons.bundle.js:3
setTimeout (async)
EuiGlobalToastList._this.dismissToast @ vendors.bundle.js:149
Ja @ vendors.bundle.js:141
invokeGuardedCallback @ vendors.bundle.js:149
invokeGuardedCallbackAndCatchFirstError @ vendors.bundle.js:149
Za @ vendors.bundle.js:141
cb @ vendors.bundle.js:141
gb @ vendors.bundle.js:141
ab @ vendors.bundle.js:141
lb @ vendors.bundle.js:141
sd @ vendors.bundle.js:149
rd @ vendors.bundle.js:141
batchedUpdates @ vendors.bundle.js:141
tc @ vendors.bundle.js:141
vd @ vendors.bundle.js:141

Steps to reproduce:

Install Elasticsearch, Kibana
Cause an event to generate a notification, example: clock generate report on dashboard page
Click on the notification 'X' to make it go away , click it twice to cause the error

screenshot from 2018-05-21 10-15-17

@liza-mae liza-mae added bug Fixes for quality problems that affect the customer experience triage_needed Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. labels May 21, 2018
@chandlerprall chandlerprall self-assigned this May 23, 2018
@chandlerprall
Copy link
Contributor

I can replicate in the Toast docs and is a bug in EUI. I'll bundle this and the EUI fix for #19285 into a special bugfix release in EUI as soon as this is fixed.

@liza-mae liza-mae removed Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc triage_needed labels May 23, 2018
@nreese
Copy link
Contributor

nreese commented May 24, 2018

closed by #19361

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. v6.3.0
Projects
None yet
Development

No branches or pull requests

3 participants