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] Allow browsing integrations in Flyout from Data Sources page #1562

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 7d29261 from #1560.

* Move add integration button to custom component

Signed-off-by: Simeon Widdis <[email protected]>

* Add flyout toggle in installed integrations table

Signed-off-by: Simeon Widdis <[email protected]>

* Fix category filters naming

Signed-off-by: Simeon Widdis <[email protected]>

* Break out integration category filter

Signed-off-by: Simeon Widdis <[email protected]>

* Repair category filtering for new install flyout

Signed-off-by: Simeon Widdis <[email protected]>

* Update testing constants for new filter handling

Signed-off-by: Simeon Widdis <[email protected]>

* Add tests for new flyout

Signed-off-by: Simeon Widdis <[email protected]>

* Move basePathLink to shared utils

Signed-off-by: Simeon Widdis <[email protected]>

---------

Signed-off-by: Simeon Widdis <[email protected]>
(cherry picked from commit 7d29261)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@Swiddis
Copy link
Collaborator

Swiddis commented Mar 18, 2024

Interdependent with #1561: Need to merge this before we can backport 1561 (or we get a gnarly merge conflict), but we need to merge both this and 1561's backport before 2.13 is cut (or we get an incomplete feature).

@Swiddis Swiddis added enhancement New feature or request integrations Used to denote items related to the Integrations project ux-integration ux related integration issues labels Mar 18, 2024
Copy link

codecov bot commented Mar 18, 2024

Codecov Report

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

Project coverage is 58.12%. Comparing base (d960b56) to head (70a3d8a).

Files Patch % Lines
...nage/integrations/installed_integrations_table.tsx 69.69% 9 Missing and 1 partial ⚠️
...rations/components/available_integration_table.tsx 50.00% 3 Missing ⚠️
common/utils/shared.ts 80.00% 1 Missing ⚠️
...ons/components/available_integration_card_view.tsx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              2.x    #1562      +/-   ##
==========================================
+ Coverage   58.09%   58.12%   +0.02%     
==========================================
  Files         362      362              
  Lines       13227    13260      +33     
  Branches     3445     3460      +15     
==========================================
+ Hits         7684     7707      +23     
- Misses       5484     5493       +9     
- Partials       59       60       +1     
Flag Coverage Δ
dashboards-observability 58.12% <66.66%> (+0.02%) ⬆️

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.

@Swiddis Swiddis merged commit 36bd75e into 2.x Mar 19, 2024
27 of 38 checks passed
@Swiddis Swiddis deleted the backport/backport-1560-to-2.x branch March 19, 2024 00:01
amsiglan pushed a commit to amsiglan/dashboards-observability that referenced this pull request Jun 7, 2024
…s page (opensearch-project#1562)

* Move add integration button to custom component

* Add flyout toggle in installed integrations table

* Fix category filters naming

* Break out integration category filter

* Repair category filtering for new install flyout

* Update testing constants for new filter handling

* Add tests for new flyout

* Move basePathLink to shared utils

---------

(cherry picked from commit 7d29261)

Signed-off-by: Simeon Widdis <[email protected]>
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 36bd75e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request integrations Used to denote items related to the Integrations project ux-integration ux related integration issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants