[7.x] Uiactions to navigate to visualize or maps (#74121)#75398
Merged
stratoula merged 1 commit intoelastic:7.xfrom Aug 19, 2020
Merged
[7.x] Uiactions to navigate to visualize or maps (#74121)#75398stratoula merged 1 commit intoelastic:7.xfrom
stratoula merged 1 commit intoelastic:7.xfrom
Conversation
* Navigate from discover to visualize with registering into a trigger * Implement the VISUALIZE_FIELD action * Implementation of the maps app trigger actions with the isCompatible functionality * clean up discover code and tile map action implementation * Add typeIsHidden on mocks * Retrieve filters and query from url state * functional test for oss and tile map * include geoshape * fix functional tests * fix types * remove unecessary dependencies * minor fixes * Remove tilemaps actios as it is going tobe deprecated * Add useEffect on discover details and move the map action to a separate folder * Retrieve map tooltips info from context * Retrieve query and filters from QueryService * Building urls with urlGenerators * replace with constants, fetch initialLayers as array * remove irrelevant comments * nice improvements * Return contextualFields for both triggers * Add getHref on actions, move capabilities to isCompatible method per action and other fixes * fix type * Fix type incompatibility after merging with master * fixes on maps plugin file after merge * remove unecessary declarations * nice improvements * Refactor maps services code to be inline with master Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Contributor
💚 Build SucceededBuild metrics@kbn/optimizer bundle module count
async chunks size
page load bundle size
To update your PR or re-run it, just comment with: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Backports the following commits to 7.x: