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] UI fixes for loading state, empty tree, added toast for error, fixed no indicies error #177

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 015cab8 from #176.

…no indicies error (#176)

* sample query button

Signed-off-by: Paul Sebastian <paulstn@amazon.com>

* added toasts for failure cases

Signed-off-by: Paul Sebastian <paulstn@amazon.com>

* added changes for error loading message, fixed initial loading screen, added badge for no indicies

Signed-off-by: sumukhswamy <sumukhhs@amazon.com>

* ui changes to curb sidebar

Signed-off-by: Paul Sebastian <paulstn@amazon.com>

* added fix for error from async query

Signed-off-by: sumukhswamy <sumukhhs@amazon.com>

* updated snapshots

Signed-off-by: Paul Sebastian <paulstn@amazon.com>

* added changes for for mocking toasts, changed table test

Signed-off-by: sumukhswamy <sumukhhs@amazon.com>

* pr requested changes

Signed-off-by: Paul Sebastian <paulstn@amazon.com>

* fix main.tsx tests

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>

---------

Signed-off-by: Paul Sebastian <paulstn@amazon.com>
Signed-off-by: sumukhswamy <sumukhhs@amazon.com>
Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>
Co-authored-by: Paul Sebastian <paulstn@amazon.com>
Co-authored-by: Shenoy Pratik <sgguruda@amazon.com>
(cherry picked from commit 015cab8)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Oct 25, 2023

Codecov Report

Merging #177 (58f5973) into 2.x (3e45ef7) will increase coverage by 3.42%.
Report is 9 commits behind head on 2.x.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              2.x     #177      +/-   ##
==========================================
+ Coverage   54.79%   58.21%   +3.42%     
==========================================
  Files          33       33              
  Lines        1356     1546     +190     
  Branches      210      270      +60     
==========================================
+ Hits          743      900     +157     
- Misses        535      622      +87     
+ Partials       78       24      -54     
Flag Coverage Δ
dashboards-query-workbench 58.21% <ø> (+3.42%) ⬆️

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

see 25 files with indirect coverage changes

@ps48 ps48 merged commit 15c51d0 into 2.x Oct 25, 2023
8 checks passed
@github-actions github-actions bot deleted the backport/backport-176-to-2.x branch October 25, 2023 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant