-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Fix/license not passed in maps #28198
Conversation
💔 Build Failed |
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.
Can you add a fix for #28119 here as well?
@thomasneirynck I'll tie in a fix in for #28119 along with my fix for #27832 per our recent conversation. Thanks! |
retest |
💔 Build Failed |
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.
lgtm with green CI
code review, ran changes in chrome
retest |
💔 Build Failed |
retest |
💔 Build Failed |
retest |
💚 Build Succeeded |
Pinging @elastic/kibana-gis |
* Add back in license wait for init routes and add license to ems client query params * Add back routes initialized check to prevent multiple initializations
Resolves #28117. For the latest ems client, the license wasn't being added to the query params. The license check has been updated to return the license uid if the license permits use of the Maps app. I've also added back in the logic originally used in #27943 before it was reverted (as it was believed license handling was being taken care of downstream, it wasn't!) which ensures routes are only initialized once.