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] [bug] address different issues with dataset selector #8667

Closed
wants to merge 1 commit into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport e23f332 from #8665.

* [bug] address some issues with dataset selector

Moved the dataset selector back to the search bar and access it by ref in the sidebar.

Avoid out of sync issue.

Signed-off-by: Kawika Avilla <[email protected]>

* update the logic for ensuring index pattern

Signed-off-by: Kawika Avilla <[email protected]>

* dont add ability to open button

Signed-off-by: Kawika Avilla <[email protected]>

* empty state but missing data set selector button

Signed-off-by: Kawika Avilla <[email protected]>

* fix tests

Signed-off-by: Kawika Avilla <[email protected]>

* add back styling

Signed-off-by: Kawika Avilla <[email protected]>

---------

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

@github-actions github-actions bot added failed changeset and removed Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry labels Oct 19, 2024
Copy link

codecov bot commented Oct 19, 2024

Codecov Report

Attention: Patch coverage is 30.23256% with 30 lines in your changes missing coverage. Please review.

Project coverage is 60.93%. Comparing base (74a69d6) to head (75e5f5a).
Report is 1 commits behind head on 2.x.

Files with missing lines Patch % Lines
...ublic/application/view_components/canvas/index.tsx 0.00% 9 Missing ⚠️
...a/public/ui/dataset_selector/advanced_selector.tsx 0.00% 5 Missing ⚠️
...components/no_index_patterns/no_index_patterns.tsx 20.00% 4 Missing ⚠️
...ta/public/ui/dataset_selector/dataset_selector.tsx 0.00% 3 Missing ⚠️
...rns/index_patterns/ensure_default_index_pattern.ts 33.33% 2 Missing ⚠️
...c/application/components/no_results/no_results.tsx 33.33% 2 Missing ⚠️
...lic/application/view_components/canvas/top_nav.tsx 33.33% 2 Missing ⚠️
...s/data/public/ui/dataset_selector/configurator.tsx 0.00% 1 Missing ⚠️
.../data_explorer/public/components/sidebar/index.tsx 80.00% 0 Missing and 1 partial ⚠️
...er/public/utils/state_management/metadata_slice.ts 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              2.x    #8667      +/-   ##
==========================================
+ Coverage   60.90%   60.93%   +0.02%     
==========================================
  Files        3787     3787              
  Lines       90341    90296      -45     
  Branches    14167    14153      -14     
==========================================
- Hits        55024    55020       -4     
+ Misses      31847    31805      -42     
- Partials     3470     3471       +1     
Flag Coverage Δ
Linux_1 29.03% <10.81%> (+0.01%) ⬆️
Linux_2 56.53% <0.00%> (-0.01%) ⬇️
Linux_3 37.72% <30.23%> (+0.02%) ⬆️
Linux_4 29.94% <0.00%> (+<0.01%) ⬆️
Windows_1 29.04% <10.81%> (+0.01%) ⬆️
Windows_2 56.48% <0.00%> (-0.01%) ⬇️
Windows_3 37.72% <30.23%> (+0.02%) ⬆️
Windows_4 29.94% <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.

@AMoo-Miki
Copy link
Collaborator

Manually backported to 2.x with #8670

@AMoo-Miki AMoo-Miki closed this Oct 19, 2024
@github-actions github-actions bot deleted the backport/backport-8665-to-2.x branch October 19, 2024 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants