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

share saved objects to workspace api #67

Merged

Conversation

Hailong-am
Copy link
Collaborator

Description

saved objects share api

Issues Resolved

Screenshot

Testing the changes

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
    • yarn test:ftr
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

@codecov-commenter
Copy link

codecov-commenter commented Jul 28, 2023

Codecov Report

Merging #67 (713463d) into workspace (c1b3a49) will decrease coverage by 0.07%.
The diff coverage is 9.33%.

@@              Coverage Diff              @@
##           workspace      #67      +/-   ##
=============================================
- Coverage      65.84%   65.77%   -0.07%     
=============================================
  Files           3339     3340       +1     
  Lines          64368    64442      +74     
  Branches       10048    10066      +18     
=============================================
+ Hits           42382    42387       +5     
- Misses         19491    19559      +68     
- Partials        2495     2496       +1     
Flag Coverage Δ
Linux_1 34.66% <ø> (ø)
Linux_3 42.72% <5.33%> (-0.09%) ⬇️
Windows_1 34.67% <ø> (ø)
Windows_2 54.43% <9.33%> (-0.14%) ⬇️
Windows_3 42.72% <5.33%> (-0.09%) ⬇️
Windows_4 34.72% <6.66%> (-0.06%) ⬇️

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

Files Changed Coverage Δ
src/core/server/index.ts 100.00% <ø> (ø)
...er/saved_objects/permission_control/client.mock.ts 100.00% <ø> (ø)
.../server/saved_objects/permission_control/client.ts 7.69% <0.00%> (-0.65%) ⬇️
...erver/saved_objects/service/lib/repository.mock.ts 100.00% <ø> (ø)
...ore/server/saved_objects/service/lib/repository.ts 84.18% <0.00%> (-6.07%) ⬇️
..._objects/workspace_saved_objects_client_wrapper.ts 6.15% <10.00%> (+0.69%) ⬆️
src/core/server/saved_objects/routes/share.ts 12.50% <12.50%> (ø)
...rver/saved_objects/service/saved_objects_client.ts 57.69% <33.33%> (-3.18%) ⬇️
src/core/server/saved_objects/routes/index.ts 100.00% <100.00%> (ø)
src/core/server/workspaces/constants.ts 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Hailong-am Hailong-am force-pushed the feature/saved_object_share_api branch from cc79490 to f4eb765 Compare August 1, 2023 04:54
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
@Hailong-am Hailong-am force-pushed the feature/saved_object_share_api branch from cde9a8a to 713463d Compare August 3, 2023 03:37
@Hailong-am Hailong-am merged commit 340bfcc into ruanyl:workspace Aug 3, 2023
45 of 47 checks passed
ruanyl pushed a commit that referenced this pull request Aug 3, 2023
* share saved objects to workspace api

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* using script for bulk update

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* filter out  pulbic saved objects when sharing

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* refactor saved object permission error

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* fix merge issue

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* permission check for target workspace

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* move source workspace existence validation to repository

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
SuZhou-Joe pushed a commit that referenced this pull request Aug 31, 2023
* share saved objects to workspace api

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* using script for bulk update

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* filter out  pulbic saved objects when sharing

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* refactor saved object permission error

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* fix merge issue

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* permission check for target workspace

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* move source workspace existence validation to repository

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
SuZhou-Joe pushed a commit that referenced this pull request Aug 31, 2023
* share saved objects to workspace api

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* using script for bulk update

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* filter out  pulbic saved objects when sharing

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* refactor saved object permission error

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* fix merge issue

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* permission check for target workspace

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* move source workspace existence validation to repository

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
ruanyl pushed a commit that referenced this pull request Sep 15, 2023
* share saved objects to workspace api

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* using script for bulk update

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* filter out  pulbic saved objects when sharing

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* refactor saved object permission error

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* fix merge issue

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* permission check for target workspace

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* move source workspace existence validation to repository

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants