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

[8.11] Fixes issue with removing spaces from related objects in share to space action (#169177) #169566

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.11:

Questions ?

Please refer to the Backport tool documentation

…ce action (elastic#169177)

closes elastic#168657

## Summary

Updates the `share to spaces` action to refrain from removing spaces
from related objects (objects referenced by the target object).

I have also updated the description of issue elastic#130562, which essentially
will replace much of the client-side implementation of this action, to
explicitly include this behavior.

### Manual Testing
- Create 2 spaces: A and B
- Add a sample data set (e.g. flight) to space A
- In Discover, create a saved query called "s1" (add a filter that uses
the sample data logs data view, and use the filter menu button)
- Go to `Stack Management->Saved` Objects and share the "s1" query to
space B
- Verify that the related data view is also shared to space B.
- Un-share the "s1" query from space B
- Verify that the related data view is still shared to space B.

### Automated Tests
-
x-pack/plugins/spaces/public/share_saved_objects_to_space/components/share_to_space_flyout_internal.test.tsx

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 4af9175)
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
spaces 173.9KB 174.9KB +982.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @jeramysoucy

@kibanamachine kibanamachine merged commit 379d9eb into elastic:8.11 Oct 23, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants