Add workspace filter/column into saved objects page #211

Hailong-am · 2023-10-09T06:04:35Z

Description

Add workspace filter into saved objects page
Add workspace column into saved objects page

workspace column

workspace filter includes default workspace

Issues Resolved

Screenshot

Testing the changes

Check List

codecov-commenter · 2023-10-09T09:00:49Z

Codecov Report

Attention: Patch coverage is 55.69620% with 35 lines in your changes are missing coverage. Please review.

Project coverage is 66.99%. Comparing base (09cfe94) to head (4804cb3).

Files	Patch %	Lines
...ment_section/objects_table/saved_objects_table.tsx	62.79%	14 Missing and 2 partials ⚠️
...d_objects_management/server/routes/scroll_count.ts	0.00%	14 Missing ⚠️
...agement_section/objects_table/components/table.tsx	77.77%	0 Missing and 2 partials ⚠️
...c/components/workspace_column/workspace_column.tsx	75.00%	2 Missing ⚠️
...s_management/public/lib/get_saved_object_counts.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@                   Coverage Diff                   @@
##           workspace-pr-integr     #211      +/-   ##
=======================================================
- Coverage                66.99%   66.99%   -0.01%     
=======================================================
  Files                     3323     3325       +2     
  Lines                    64460    64535      +75     
  Branches                 10387    10406      +19     
=======================================================
+ Hits                     43188    43233      +45     
- Misses                   18744    18770      +26     
- Partials                  2528     2532       +4

Flag	Coverage Δ
Linux_	`66.32% <55.69%> (?)`
_1	`32.22% <61.53%> (+0.03%)`	⬆️
_2	`55.39% <100.00%> (+<0.01%)`	⬆️
_3	`43.31% <100.00%> (-0.01%)`	⬇️
_4	`35.14% <52.85%> (+0.05%)`	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

...lugins/saved_objects_management/public/management_section/objects_table/components/table.tsx

...aved_objects_management/public/management_section/objects_table/saved_objects_table.test.tsx

...ins/saved_objects_management/public/management_section/objects_table/saved_objects_table.tsx

Signed-off-by: Hailong Cui <[email protected]> fix failed test case Signed-off-by: Hailong Cui <[email protected]> move workspace column to its own folder Signed-off-by: Hailong Cui <[email protected]>

Signed-off-by: Hailong Cui <[email protected]> fix test case Signed-off-by: Hailong Cui <[email protected]> add test case Signed-off-by: Hailong Cui <[email protected]> remove hide import Signed-off-by: Hailong Cui <[email protected]>

SuZhou-Joe · 2024-03-08T10:05:35Z

...aved_objects_management/public/management_section/objects_table/saved_objects_table.test.tsx

+import { WorkspaceObject } from 'opensearch-dashboards/public';
+import { DEFAULT_WORKSPACE_ID } from '../../../../../core/public';
+import { TableProps } from './components/table';
+import { waitFor } from '@testing-library/dom';


These lines should be linted when commit(Absolute import should move before relative import), could you please take a look to see if the lint take effect when you do git commit?

SuZhou-Joe · 2024-03-08T10:10:52Z

src/core/utils/constants.ts

@@ -8,3 +8,5 @@ export const WORKSPACE_TYPE = 'workspace';
 export const WORKSPACE_PATH_PREFIX = '/w';

 export const PUBLIC_WORKSPACE_ID = 'public';
+
+export const DEFAULT_WORKSPACE_ID = 'default';


In 2.13 I thought default workspace is a virtual workspace? Could we add some comment on it as at first I thought it is a new physical workspace.

sounds good, comments added.

SuZhou-Joe · 2024-03-08T10:20:33Z

src/plugins/workspace/public/plugin.test.ts

@@ -9,6 +9,7 @@ import { applicationServiceMock, chromeServiceMock, coreMock } from '../../../co
 import { WorkspacePlugin } from './plugin';
 import { WORKSPACE_FATAL_ERROR_APP_ID, WORKSPACE_OVERVIEW_APP_ID } from '../common/constants';
 import { Observable, Subscriber } from 'rxjs';
+import { savedObjectsManagementPluginMock } from 'src/plugins/saved_objects_management/public/mocks';


Perhaps we need to change to a relative path.

thanks, changed to relative import.

Signed-off-by: Hailong Cui <[email protected]>

* Add workspace column/filter into saved objects page Signed-off-by: Hailong Cui <[email protected]> fix failed test case Signed-off-by: Hailong Cui <[email protected]> move workspace column to its own folder Signed-off-by: Hailong Cui <[email protected]> * default workspace Signed-off-by: Hailong Cui <[email protected]> fix test case Signed-off-by: Hailong Cui <[email protected]> add test case Signed-off-by: Hailong Cui <[email protected]> remove hide import Signed-off-by: Hailong Cui <[email protected]> * address review comments Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]>

…ensearch-project#6458) * Add workspace filter into saved objects page (#211) * Add workspace column/filter into saved objects page Signed-off-by: Hailong Cui <[email protected]> fix failed test case Signed-off-by: Hailong Cui <[email protected]> move workspace column to its own folder Signed-off-by: Hailong Cui <[email protected]> * default workspace Signed-off-by: Hailong Cui <[email protected]> fix test case Signed-off-by: Hailong Cui <[email protected]> add test case Signed-off-by: Hailong Cui <[email protected]> remove hide import Signed-off-by: Hailong Cui <[email protected]> * address review comments Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> * replace default workspace with public workspace (#322) Signed-off-by: Hailong Cui <[email protected]> * Add workspace filter Signed-off-by: Hailong Cui <[email protected]> * revert query params change Signed-off-by: Hailong Cui <[email protected]> * udpate test case name to more clear Signed-off-by: Hailong Cui <[email protected]> * support public workspace Signed-off-by: Hailong Cui <[email protected]> * Add changelog Signed-off-by: Hailong Cui <[email protected]> * update fetchExportByTypeAndSearch parameter Co-authored-by: SuZhou-Joe <[email protected]> Signed-off-by: Hailong Cui <[email protected]> * remove virtrual public workspace Signed-off-by: Hailong Cui <[email protected]> * update comments Signed-off-by: Hailong Cui <[email protected]> * fix public workspace query when permission is not enabled Signed-off-by: Hailong Cui <[email protected]> * home dashboards/listing page only show public workspace data Signed-off-by: Hailong Cui <[email protected]> * Add more test case Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> Signed-off-by: SuZhou-Joe <[email protected]> Co-authored-by: SuZhou-Joe <[email protected]>

…t page (#6… (#334) * [Workspace] Add Workspace filter to saved objects management page (opensearch-project#6458) * Add workspace filter into saved objects page (#211) * Add workspace column/filter into saved objects page Signed-off-by: Hailong Cui <[email protected]> fix failed test case Signed-off-by: Hailong Cui <[email protected]> move workspace column to its own folder Signed-off-by: Hailong Cui <[email protected]> * default workspace Signed-off-by: Hailong Cui <[email protected]> fix test case Signed-off-by: Hailong Cui <[email protected]> add test case Signed-off-by: Hailong Cui <[email protected]> remove hide import Signed-off-by: Hailong Cui <[email protected]> * address review comments Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> * replace default workspace with public workspace (#322) Signed-off-by: Hailong Cui <[email protected]> * Add workspace filter Signed-off-by: Hailong Cui <[email protected]> * revert query params change Signed-off-by: Hailong Cui <[email protected]> * udpate test case name to more clear Signed-off-by: Hailong Cui <[email protected]> * support public workspace Signed-off-by: Hailong Cui <[email protected]> * Add changelog Signed-off-by: Hailong Cui <[email protected]> * update fetchExportByTypeAndSearch parameter Co-authored-by: SuZhou-Joe <[email protected]> Signed-off-by: Hailong Cui <[email protected]> * remove virtrual public workspace Signed-off-by: Hailong Cui <[email protected]> * update comments Signed-off-by: Hailong Cui <[email protected]> * fix public workspace query when permission is not enabled Signed-off-by: Hailong Cui <[email protected]> * home dashboards/listing page only show public workspace data Signed-off-by: Hailong Cui <[email protected]> * Add more test case Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> Signed-off-by: SuZhou-Joe <[email protected]> Co-authored-by: SuZhou-Joe <[email protected]> * fix empty workspace Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> Signed-off-by: SuZhou-Joe <[email protected]> Co-authored-by: SuZhou-Joe <[email protected]>

…ensearch-project#6458) (opensearch-project#6580) * Add workspace filter into saved objects page (#211) * Add workspace column/filter into saved objects page Signed-off-by: Hailong Cui <[email protected]> fix failed test case Signed-off-by: Hailong Cui <[email protected]> move workspace column to its own folder Signed-off-by: Hailong Cui <[email protected]> * default workspace Signed-off-by: Hailong Cui <[email protected]> fix test case Signed-off-by: Hailong Cui <[email protected]> add test case Signed-off-by: Hailong Cui <[email protected]> remove hide import Signed-off-by: Hailong Cui <[email protected]> * address review comments Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> * replace default workspace with public workspace (#322) Signed-off-by: Hailong Cui <[email protected]> * Add workspace filter Signed-off-by: Hailong Cui <[email protected]> * revert query params change Signed-off-by: Hailong Cui <[email protected]> * udpate test case name to more clear Signed-off-by: Hailong Cui <[email protected]> * support public workspace Signed-off-by: Hailong Cui <[email protected]> * Add changelog Signed-off-by: Hailong Cui <[email protected]> * update fetchExportByTypeAndSearch parameter Co-authored-by: SuZhou-Joe <[email protected]> Signed-off-by: Hailong Cui <[email protected]> * remove virtrual public workspace Signed-off-by: Hailong Cui <[email protected]> * update comments Signed-off-by: Hailong Cui <[email protected]> * fix public workspace query when permission is not enabled Signed-off-by: Hailong Cui <[email protected]> * home dashboards/listing page only show public workspace data Signed-off-by: Hailong Cui <[email protected]> * Add more test case Signed-off-by: Hailong Cui <[email protected]> --------- Signed-off-by: Hailong Cui <[email protected]> Signed-off-by: SuZhou-Joe <[email protected]> Co-authored-by: SuZhou-Joe <[email protected]> (cherry picked from commit 9408e49) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> # Conflicts: # CHANGELOG.md Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

github-actions bot added the seasoned-contributor label Oct 9, 2023

Hailong-am changed the title ~~Add workspace filter into saved objects page~~ Add workspace filter/column into saved objects page Oct 9, 2023

Hailong-am marked this pull request as draft October 10, 2023 07:21

Hailong-am force-pushed the workspace_filter branch 2 times, most recently from 1996ecc to d5d3d4f Compare October 17, 2023 03:29

Hailong-am marked this pull request as ready for review October 17, 2023 03:31

Hailong-am force-pushed the workspace_filter branch 2 times, most recently from 7de3949 to 25b62bd Compare October 19, 2023 05:37

Hailong-am requested a review from SuZhou-Joe October 26, 2023 02:52

wanglam reviewed Oct 26, 2023

View reviewed changes

Hailong-am force-pushed the workspace_filter branch from 94627b0 to 2b0d341 Compare October 27, 2023 11:57

wanglam force-pushed the workspace-pr-integr branch from 8ff9e88 to 1a52b29 Compare February 26, 2024 07:16

Hailong-am force-pushed the workspace_filter branch 4 times, most recently from 7ea5621 to f54318b Compare March 1, 2024 07:34

Hailong-am force-pushed the workspace_filter branch 2 times, most recently from 2243734 to 6d473d8 Compare March 8, 2024 03:38

Hailong-am added 2 commits March 8, 2024 18:01

Add workspace column/filter into saved objects page

568d414

Signed-off-by: Hailong Cui <[email protected]> fix failed test case Signed-off-by: Hailong Cui <[email protected]> move workspace column to its own folder Signed-off-by: Hailong Cui <[email protected]>

default workspace

023fa9b

Signed-off-by: Hailong Cui <[email protected]> fix test case Signed-off-by: Hailong Cui <[email protected]> add test case Signed-off-by: Hailong Cui <[email protected]> remove hide import Signed-off-by: Hailong Cui <[email protected]>

Hailong-am force-pushed the workspace_filter branch from 448fdb2 to 023fa9b Compare March 8, 2024 10:01

SuZhou-Joe approved these changes Mar 8, 2024

View reviewed changes

address review comments

4804cb3

Signed-off-by: Hailong Cui <[email protected]>

Hailong-am merged commit 58f8ea4 into ruanyl:workspace-pr-integr Mar 11, 2024
44 of 45 checks passed

Hailong-am mentioned this pull request Apr 18, 2024

[Backport][Workspace] Add Workspace filter to saved objects management page (#6… #334

Merged

7 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add workspace filter/column into saved objects page #211

Add workspace filter/column into saved objects page #211

Hailong-am commented Oct 9, 2023 •

edited

Loading

codecov-commenter commented Oct 9, 2023 •

edited

Loading

SuZhou-Joe Mar 8, 2024

SuZhou-Joe Mar 8, 2024

Hailong-am Mar 8, 2024

SuZhou-Joe Mar 8, 2024

Hailong-am Mar 8, 2024

Add workspace filter/column into saved objects page #211

Add workspace filter/column into saved objects page #211

Conversation

Hailong-am commented Oct 9, 2023 • edited Loading

Description

Issues Resolved

Screenshot

Testing the changes

Check List

codecov-commenter commented Oct 9, 2023 • edited Loading

Codecov Report

SuZhou-Joe Mar 8, 2024

Choose a reason for hiding this comment

SuZhou-Joe Mar 8, 2024

Choose a reason for hiding this comment

Hailong-am Mar 8, 2024

Choose a reason for hiding this comment

SuZhou-Joe Mar 8, 2024

Choose a reason for hiding this comment

Hailong-am Mar 8, 2024

Choose a reason for hiding this comment

Hailong-am commented Oct 9, 2023 •

edited

Loading

codecov-commenter commented Oct 9, 2023 •

edited

Loading