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] [Feature] Associated objects searchbar filters #1492

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 1b005d4 from #1474.

* Add associated objects filters for databases and accelerations

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Fix both filters

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Fix some UI details

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Finalize the UI details

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Fix the acceleration filter type

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Switch to lodash

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Enable multiple filter selections 1

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Make the filters logic into OR

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Fix some comments

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Switch back to JS instead of using lodash

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Update the snapshots

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Remove the inline style

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Set up an util file

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Add a todo comment for dummy data

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Update the snapshots

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Address some minor comments

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Assign types for obj and clauses

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Setup the consts and replace them

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Finish an i18n example

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Finish the i18n for AO tab

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* i18n done

Signed-off-by: Ryan Liang <jiallian@amazon.com>

* Add jest test

Signed-off-by: Ryan Liang <jiallian@amazon.com>

---------

Signed-off-by: Ryan Liang <jiallian@amazon.com>
(cherry picked from commit 1b005d4)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link

codecov bot commented Mar 6, 2024

Codecov Report

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

Project coverage is 54.73%. Comparing base (e650fcd) to head (e17811f).
Report is 2 commits behind head on 2.x.

Files Patch % Lines
...nage/associated_objects/associated_objects_tab.tsx 56.52% 20 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              2.x    #1492      +/-   ##
==========================================
+ Coverage   54.64%   54.73%   +0.08%     
==========================================
  Files         340      341       +1     
  Lines       12047    12099      +52     
  Branches     3110     3121      +11     
==========================================
+ Hits         6583     6622      +39     
- Misses       5410     5424      +14     
+ Partials       54       53       -1     
Flag Coverage Δ
dashboards-observability 54.73% <64.28%> (+0.08%) ⬆️

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.

@RyanL1997 RyanL1997 added the enhancement New feature or request label Mar 6, 2024
@ps48 ps48 merged commit c67bcb5 into 2.x Mar 6, 2024
15 of 34 checks passed
@github-actions github-actions bot deleted the backport/backport-1474-to-2.x branch March 6, 2024 23:59
amsiglan pushed a commit to amsiglan/dashboards-observability that referenced this pull request Jun 7, 2024
…) (opensearch-project#1492)

* Add associated objects filters for databases and accelerations

* Fix both filters

* Fix some UI details

* Finalize the UI details

* Fix the acceleration filter type

* Switch to lodash

* Enable multiple filter selections 1

* Make the filters logic into OR

* Fix some comments

* Switch back to JS instead of using lodash

* Update the snapshots

* Remove the inline style

* Set up an util file

* Add a todo comment for dummy data

* Update the snapshots

* Address some minor comments

* Assign types for obj and clauses

* Setup the consts and replace them

* Finish an i18n example

* Finish the i18n for AO tab

* i18n done

* Add jest test

---------

(cherry picked from commit 1b005d4)

Signed-off-by: Ryan Liang <jiallian@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit c67bcb5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants