Skip to content

[Backport 2.x] [Workspace] Support getting workspace client from coreStart#7797

Merged
SuZhou-Joe merged 1 commit into2.xfrom
backport/backport-7501-to-2.x
Aug 22, 2024
Merged

[Backport 2.x] [Workspace] Support getting workspace client from coreStart#7797
SuZhou-Joe merged 1 commit into2.xfrom
backport/backport-7501-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport ae7cfa6 from #7501.

* Support to get workspace client from coreStart

Signed-off-by: yubonluo <yubonluo@amazon.com>

* Changeset file for PR #7501 created/updated

* Support to get workspace client from coreStart

Signed-off-by: yubonluo <yubonluo@amazon.com>

* fix test error

Signed-off-by: yubonluo <yubonluo@amazon.com>

* delete irrelevant modification

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>

---------

Signed-off-by: yubonluo <yubonluo@amazon.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
Co-authored-by: Yulong Ruan <ruanyl@amazon.com>
(cherry picked from commit ae7cfa6)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

❌ Invalid Changelog Heading

The '## Changelog' heading in your PR description is either missing or malformed. Please make sure that your PR description includes a '## Changelog' heading with proper spelling, capitalization, spacing, and Markdown syntax.

@codecov
Copy link

codecov bot commented Aug 22, 2024

Codecov Report

Attention: Patch coverage is 94.73684% with 1 line in your changes missing coverage. Please review.

Project coverage is 63.88%. Comparing base (f2faa0a) to head (4985479).
Report is 5 commits behind head on 2.x.

Files Patch % Lines
src/plugins/workspace/public/workspace_client.ts 80.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##              2.x    #7797   +/-   ##
=======================================
  Coverage   63.88%   63.88%           
=======================================
  Files        3658     3657    -1     
  Lines       81196    81208   +12     
  Branches    12934    12935    +1     
=======================================
+ Hits        51871    51882   +11     
  Misses      26153    26153           
- Partials     3172     3173    +1     
Flag Coverage Δ
Linux_1 30.09% <31.57%> (+<0.01%) ⬆️
Linux_2 55.99% <100.00%> (+0.01%) ⬆️
Linux_3 40.47% <7.69%> (-0.02%) ⬇️
Linux_4 31.45% <76.92%> (+<0.01%) ⬆️
Windows_1 30.10% <31.57%> (+<0.01%) ⬆️
Windows_2 55.94% <100.00%> (+0.01%) ⬆️
Windows_3 40.49% <7.69%> (-0.01%) ⬇️
Windows_4 31.45% <76.92%> (+<0.01%) ⬆️

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.

@SuZhou-Joe SuZhou-Joe merged commit 7fba6ac into 2.x Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autocut Skip the changelog verification check on backports failed changeset seasoned-contributor v2.17.0 workspace

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants