-
Notifications
You must be signed in to change notification settings - Fork 867
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
[Workspace] Delete the virtual global workspace #7165
[Workspace] Delete the virtual global workspace #7165
Conversation
Signed-off-by: yubonluo <yubonluo@amazon.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7165 +/- ##
=======================================
Coverage 67.54% 67.55%
=======================================
Files 3469 3469
Lines 68502 68478 -24
Branches 11140 11130 -10
=======================================
- Hits 46273 46263 -10
+ Misses 19524 19512 -12
+ Partials 2705 2703 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: yubonluo <yubonluo@amazon.com>
…enSearch-Dashboards into 2.16-delete-global-workspace
Signed-off-by: yubonluo <yubonluo@amazon.com>
…oards into 2.16-delete-global-workspace
…enSearch-Dashboards into 2.16-delete-global-workspace
src/plugins/workspace/server/saved_objects/workspace_saved_objects_client_wrapper.ts
Outdated
Show resolved
Hide resolved
Signed-off-by: yubonluo <yubonluo@amazon.com>
Signed-off-by: yubonluo <yubonluo@amazon.com>
Signed-off-by: yubonluo <yubonluo@amazon.com>
Signed-off-by: yubonluo <yubonluo@amazon.com>
…enSearch-Dashboards into 2.16-delete-global-workspace
src/plugins/workspace/server/saved_objects/workspace_saved_objects_client_wrapper.ts
Outdated
Show resolved
Hide resolved
I like these changes, after removing the special "artificial" workspace, the code smells much better 😃 |
Signed-off-by: yubonluo <yubonluo@amazon.com>
* Delete the virtual global workspace Signed-off-by: yubonluo <yubonluo@amazon.com> * Changeset file for PR #7165 created/updated * optimize the code Signed-off-by: yubonluo <yubonluo@amazon.com> * optimize the code Signed-off-by: yubonluo <yubonluo@amazon.com> * optimize the code Signed-off-by: yubonluo <yubonluo@amazon.com> * optimize the code Signed-off-by: yubonluo <yubonluo@amazon.com> * delete the useless code Signed-off-by: yubonluo <yubonluo@amazon.com> * delete the useless code Signed-off-by: yubonluo <yubonluo@amazon.com> --------- Signed-off-by: yubonluo <yubonluo@amazon.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> (cherry picked from commit 20cefab) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Delete the virtual global workspace * Changeset file for PR #7165 created/updated * optimize the code * optimize the code * optimize the code * optimize the code * delete the useless code * delete the useless code --------- (cherry picked from commit 20cefab) Signed-off-by: yubonluo <yubonluo@amazon.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
Description
Delete the virtual global workspace.
When workspace feature flag is enabled and user has not created any workspace, only OSD admin user can see any legacy saved object (not belonging to any workspace) on the home page. Non OSD admin users should not see the legacy saved objects if they can not access any workspace.
Issues Resolved
#7095
Screenshot
Testing the changes
Changelog
Check List
yarn test:jest
yarn test:jest_integration