-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Canvas] Fixes Storybook for DatasourceComponent is crashing. #110180
[Canvas] Fixes Storybook for DatasourceComponent is crashing. #110180
Conversation
Pinging @elastic/kibana-presentation (Team:Presentation) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Left one small nit that needs to be looked at and addressed
@@ -20,7 +20,7 @@ import { | |||
import { isEqual } from 'lodash'; | |||
import { i18n } from '@kbn/i18n'; | |||
|
|||
import { getDefaultIndex } from '../../lib/es_service'; | |||
import { getDefaultIndex } from '../../../public/lib/es_service'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason this import had to change? Please leave it as it was if not necessary to change
@elasticmachine merge upstream |
💚 Build SucceededMetrics [docs]
History
To update your PR or re-run it, just comment with: cc @Kunzetsov |
…c#110180) * Added mock for `es_service`. Co-authored-by: Kibana Machine <[email protected]>
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
… (#110397) * Added mock for `es_service`. Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Yaroslav Kuznietsov <[email protected]>
Closes: #105190.
Added
mock
fores_service
to prevent calling not existing function.Before:
After: