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

[Backport 2.x] Add set default datasource #6200

Merged
merged 1 commit into from
Mar 19, 2024
Merged

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport d2347ca from #6186.

* 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]>
(cherry picked from commit d2347ca)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
Copy link

codecov bot commented Mar 19, 2024

Codecov Report

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

Project coverage is 67.19%. Comparing base (52fca45) to head (c9bb5a8).

Files Patch % Lines
...rce/components/edit_form/edit_data_source_form.tsx 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##              2.x    #6200   +/-   ##
=======================================
  Coverage   67.18%   67.19%           
=======================================
  Files        3332     3332           
  Lines       64421    64435   +14     
  Branches    10286    10289    +3     
=======================================
+ Hits        43281    43294   +13     
  Misses      18642    18642           
- Partials     2498     2499    +1     
Flag Coverage Δ
Linux_1 35.07% <ø> (ø)
Linux_2 55.28% <ø> (ø)
Linux_3 44.76% <92.85%> (+0.02%) ⬆️
Linux_4 35.35% <ø> (ø)
Windows_1 35.10% <ø> (ø)
Windows_2 55.24% <ø> (ø)
Windows_3 44.76% <92.85%> (+0.02%) ⬆️
Windows_4 35.35% <ø> (ø)

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.

@ananzh ananzh added the v2.13.0 label Mar 19, 2024
@ZilongX ZilongX merged commit 5a7d584 into 2.x Mar 19, 2024
91 of 92 checks passed
@ZilongX ZilongX deleted the backport/backport-6186-to-2.x branch March 19, 2024 18:46
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 first-time-contributor multiple datasource multiple datasource project v2.13.0
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants