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

Revert "feat: make the pr smaller" #16

Closed
wants to merge 20 commits into from

Conversation

SuZhou-Joe
Copy link
Owner

This reverts commit 9b66a28.

Description

Issues Resolved

Screenshot

Testing the changes

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
  • 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 Mar 14, 2024

Codecov Report

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

Project coverage is 67.31%. Comparing base (05abf5e) to head (fc570f3).

Files Patch % Lines
...erver/saved_objects/import/create_saved_objects.ts 68.88% 10 Missing and 4 partials ⚠️
src/plugins/workspace/public/plugin.ts 68.42% 5 Missing and 1 partial ⚠️
src/core/server/saved_objects/import/utils.ts 90.24% 2 Missing and 2 partials ⚠️
..._source/server/routes/fetch_data_source_version.ts 81.25% 3 Missing ⚠️
...d_objects/import/check_conflict_for_data_source.ts 80.00% 0 Missing and 2 partials ⚠️
.../server/routes/data_source_connection_validator.ts 77.77% 1 Missing and 1 partial ⚠️
.../data_source_management/public/components/utils.ts 0.00% 2 Missing ⚠️
src/plugins/vis_type_vega/server/utils.ts 95.23% 1 Missing and 1 partial ⚠️
...e_vega/server/vega_visualization_client_wrapper.ts 93.33% 1 Missing and 1 partial ⚠️
.../plugins/application_config/server/routes/index.ts 88.88% 1 Missing ⚠️
... and 2 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
+ Coverage   67.24%   67.31%   +0.07%     
==========================================
  Files        3344     3350       +6     
  Lines       64792    64993     +201     
  Branches    10427    10468      +41     
==========================================
+ Hits        43568    43752     +184     
+ Misses      18683    18674       -9     
- Partials     2541     2567      +26     
Flag Coverage Δ
Linux_1 31.75% <48.45%> (+0.10%) ⬆️
Linux_2 55.55% <79.16%> (+0.10%) ⬆️
Linux_3 44.67% <35.48%> (?)
Linux_4 35.02% <5.82%> (-0.04%) ⬇️
Windows_1 31.77% <48.45%> (+0.10%) ⬆️
Windows_2 55.52% <79.16%> (+0.10%) ⬆️
Windows_3 44.68% <35.48%> (-0.02%) ⬇️
Windows_4 35.02% <5.82%> (-0.04%) ⬇️

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 force-pushed the feature/validate-workspace-id-when-setup branch 3 times, most recently from 52672e1 to 9a9c39c Compare March 15, 2024 09:11
ashwin-pc and others added 18 commits March 18, 2024 13:12
…lugin (opensearch-project#6185)

* remove obsolete snapshots

Signed-off-by: Lu Yu <[email protected]>

* add change log

Signed-off-by: Lu Yu <[email protected]>

---------

Signed-off-by: Lu Yu <[email protected]>
…-project#6164)

Application config plugin allows external plugin to register a different storage other than the default OpenSearch. The external plugin may need some additional information about the request headers when handling the requests.

* Allow extra information in the application config calls

Signed-off-by: Tianle Huang <[email protected]>

* only expose headers

Signed-off-by: Tianle Huang <[email protected]>

* update jsdoc

Signed-off-by: Tianle Huang <[email protected]>

* add more logs

Signed-off-by: Tianle Huang <[email protected]>

* update change log

Signed-off-by: Tianle Huang <[email protected]>

* update readme

Signed-off-by: Tianle Huang <[email protected]>

* update readme

Signed-off-by: Tianle Huang <[email protected]>

---------

Signed-off-by: Tianle Huang <[email protected]>
…-project#6170)

* [Discover] options button in canvas to toggle legacy mode

Removes top nav bar link and provides a button option.

Issue:
n/a

Signed-off-by: Kawika Avilla <[email protected]>
…ject#6178)

* add version number to newly created datasource object

Signed-off-by: Zilong Xia <[email protected]>

* update attribute key from version to dataSourceVersion

Signed-off-by: Zilong Xia <[email protected]>

* add support and test coverage for custom auth type

Signed-off-by: Zilong Xia <[email protected]>

* fix ciGroup3 test case suite Create Datasource Wizard

Signed-off-by: Zilong Xia <[email protected]>

---------

Signed-off-by: Zilong Xia <[email protected]>
Signed-off-by: ZilongX <[email protected]>
* Add set default datasource

Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>

* Fix typo

Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>

* change on this.props.isDefault

Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>

* add unit test

Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>

* set data_source to false

Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>

* add more unit test

Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>

* fix lint error

Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>

* edit one more unit test

Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>

* Fix another typo

Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>

---------

Signed-off-by: Yuanqi(Ella) Zhu <[email protected]>
Co-authored-by: ZilongX <[email protected]>
opensearch-project#6175)

* use filter function before rendering

Signed-off-by: Lu Yu <[email protected]>

* add change log

Signed-off-by: Lu Yu <[email protected]>

---------

Signed-off-by: Lu Yu <[email protected]>
Co-authored-by: ZilongX <[email protected]>
…rch-project#6201)

* updated follow redirects

Signed-off-by: Ashwin P Chandran <[email protected]>

* Adds changelog

Signed-off-by: Ashwin P Chandran <[email protected]>

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
…t#6123)

* Add MDS support for Vega

Signed-off-by: Huy Nguyen <[email protected]>

* Refactor field to data_source_id

Signed-off-by: Huy Nguyen <[email protected]>

* Add to CHANGELOG.md

Signed-off-by: Huy Nguyen <[email protected]>

* Added test cases and renamed field to use data_source_name

Signed-off-by: Huy Nguyen <[email protected]>

* Add prefix datasource name test case and add example in default hjson

Signed-off-by: Huy Nguyen <[email protected]>

* Move CHANGELOG to appropriate section

Signed-off-by: Huy Nguyen <[email protected]>

* Increased test coverage of search() method

Signed-off-by: Huy Nguyen <[email protected]>

* Add test cases for util function

Signed-off-by: Huy Nguyen <[email protected]>

* Add util function to modify Vega Spec

Signed-off-by: Huy Nguyen <[email protected]>

* Add method to verify Vega saved object type

Signed-off-by: Huy Nguyen <[email protected]>

* Add import saved object support for Vega

Signed-off-by: Huy Nguyen <[email protected]>

* Add unit tests for Vega objects in create and conflict modes

Signed-off-by: Huy Nguyen <[email protected]>

* Refactored utils test file

Signed-off-by: Huy Nguyen <[email protected]>

* Add to CHANGELOG

Signed-off-by: Huy Nguyen <[email protected]>

* Use bulkget instead of single get

Signed-off-by: Huy Nguyen <[email protected]>

* Add datasource references to the specs

Signed-off-by: Huy Nguyen <[email protected]>

* Fix bootstrap errors

Signed-off-by: Huy Nguyen <[email protected]>

* Add edge case where title is potentially undefined

Signed-off-by: Huy Nguyen <[email protected]>

* Address PR comments

Signed-off-by: Huy Nguyen <[email protected]>

* Add more test coverage for checking conflict

Signed-off-by: Huy Nguyen <[email protected]>

* Fix unit test

Signed-off-by: Huy Nguyen <[email protected]>

---------

Signed-off-by: Huy Nguyen <[email protected]>
* add @xinruiba as a maintainer

Signed-off-by: Flyingliuhub <[email protected]>

* update PR Id

Signed-off-by: Flyingliuhub <[email protected]>

---------

Signed-off-by: Flyingliuhub <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
@SuZhou-Joe SuZhou-Joe force-pushed the feature/validate-workspace-id-when-setup branch from cf4c84c to 8350bef Compare March 20, 2024 03:08
SuZhou-Joe and others added 2 commits March 20, 2024 11:43
Co-authored-by: Yulong Ruan <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
@SuZhou-Joe SuZhou-Joe closed this Jul 19, 2024
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.