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] [Discover - datasource selector] Add extension group title and modal #6359

Merged
merged 1 commit into from
Apr 9, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport a0eaf84 from #5815.

…5815)

* add log explorer re-directon modal

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

* adjustments to comments

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

* add one missing i18n

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

* add redirection text to group title

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

* include changes in changelog

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

* remove redundent title addition and unnecessary modal toggle functions

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

* remove one comment

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

* add i18n

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

* add unit tests for modal

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

* test id change

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

* add devDependencies for tests

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

* use open confirm api and move mock file to discover mock folder

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

* remove unused type

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

* remove modal for log explorer redirection

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

* modify changelog

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

* remove modal test

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

* remove one modal related test

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

---------

Signed-off-by: Eric <[email protected]>
(cherry picked from commit a0eaf84)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
Copy link

codecov bot commented Apr 6, 2024

Codecov Report

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

Project coverage is 67.39%. Comparing base (4b5d845) to head (36cf638).

Files Patch % Lines
...rces/datasource_selector/datasource_selectable.tsx 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              2.x    #6359      +/-   ##
==========================================
+ Coverage   67.34%   67.39%   +0.04%     
==========================================
  Files        3344     3345       +1     
  Lines       64887    64898      +11     
  Branches    10398    10399       +1     
==========================================
+ Hits        43701    43738      +37     
+ Misses      18657    18623      -34     
- Partials     2529     2537       +8     
Flag Coverage Δ
Linux_1 31.30% <0.00%> (-0.01%) ⬇️
Linux_2 55.39% <ø> (ø)
Linux_3 45.19% <92.30%> (+0.08%) ⬆️
Linux_4 35.20% <0.00%> (-0.01%) ⬇️
Windows_1 31.32% <0.00%> (-0.01%) ⬇️
Windows_2 55.35% <ø> (ø)
Windows_3 45.21% <92.30%> (+0.09%) ⬆️
Windows_4 35.20% <0.00%> (-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 89ba73b into 2.x Apr 9, 2024
88 of 89 checks passed
@github-actions github-actions bot deleted the backport/backport-5815-to-2.x branch April 9, 2024 02:28
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 repeat-contributor v2.14.0
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants