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
[maps] show actionable message when term joins have no matches #105161
Conversation
Pinging @elastic/kibana-gis (Team:Geo) |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
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.
This is a real nice UX-improvement. It's something the region-maps used to have :)
Any thoughts on how to add test coverage? e.g. tiled_vector_layer
and vector_tile_layer
both use VectorLayer#syncData
with mock-SyncContext. imho, could introduce something similar for VectorLayer#syncData
.
x-pack/plugins/maps/public/classes/layers/vector_layer/vector_layer.tsx
Outdated
Show resolved
Hide resolved
|
How about moving the reason for the error closer to the top: Unable to perform term join. The left field does not match the right field. Left field: iso_3166-2 has values US-MI, UsVI..... Right field: |
@elasticmachine merge upstream |
💚 Build SucceededMetrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
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.
real nice win for the user 💯
…ic#105161) * [maps] show actionable message when term joins have no matches * include message about showing first 10 * fix formatting * review feedback * review feedback * move performInnerJoins into its own function * lint * unit test Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
…) (#105588) * [maps] show actionable message when term joins have no matches * include message about showing first 10 * fix formatting * review feedback * review feedback * move performInnerJoins into its own function * lint * unit test Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Nathan Reese <reese.nathan@gmail.com>
fixes #105103