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] Refactor for message action bar and add metrics for alerting summary #320

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport db5155b from #304.

…304)

* Add message action module

Signed-off-by: Sihan He <[email protected]>

* Finish basic function of feedback buttons

Signed-off-by: Sihan He <[email protected]>

* Add test cases and fix bugs

Signed-off-by: Sihan He <[email protected]>

* Refactor the code to deal with core loss and fix the regenerate bug

Signed-off-by: Sihan He <[email protected]>

* Update CHANGELOG.md

Signed-off-by: Sihan He <[email protected]>

* Update for smaller buttons

Signed-off-by: Sihan He <[email protected]>

* Update for ui-metric

Signed-off-by: Sihan He <[email protected]>

* Support insight with RAG

Signed-off-by: Heng Qian <[email protected]>

* Add change in CHANGELOG.md

Signed-off-by: Heng Qian <[email protected]>

* Put footer in the same panel with content to match UX design

Signed-off-by: Heng Qian <[email protected]>

* Refine alert prompt

Signed-off-by: Heng Qian <[email protected]>

* set CSS scrollbar-width to thin

Signed-off-by: Heng Qian <[email protected]>

* Hide insight agent id from front-end

Signed-off-by: Heng Qian <[email protected]>

* Change summary agent config id

Signed-off-by: Heng Qian <[email protected]>

* Address comments

Signed-off-by: Heng Qian <[email protected]>

* Incorporate message action buttons into alerting summary

Signed-off-by: Sihan He <[email protected]>

* Add hover tips and rearrange buttons

Signed-off-by: Sihan He <[email protected]>

* Fix UT

Signed-off-by: Heng Qian <[email protected]>

* Add ui-metric for alerting summary

Signed-off-by: Sihan He <[email protected]>

* fix testing bugs

Signed-off-by: Sihan He <[email protected]>

* Change agent execute API

Signed-off-by: Heng Qian <[email protected]>

* Remove prompt from node JS server

Signed-off-by: Heng Qian <[email protected]>

* Replace CSS with component property

Signed-off-by: Heng Qian <[email protected]>

* Save the status when switch between summary and insight

Signed-off-by: Sihan He <[email protected]>

* Add tests for metrics

Signed-off-by: Sihan He <[email protected]>

* Use usageCollectionPluginMock in tests

Signed-off-by: Sihan He <[email protected]>

* Refactor reportMetric as util funciton

Signed-off-by: Sihan He <[email protected]>

* Remove telementry plugin

Signed-off-by: Sihan He <[email protected]>

* Fix metric type

Signed-off-by: Sihan He <[email protected]>

---------

Signed-off-by: Sihan He <[email protected]>
Signed-off-by: 000FLMS <[email protected]>
Signed-off-by: Heng Qian <[email protected]>
Co-authored-by: Heng Qian <[email protected]>
(cherry picked from commit db5155b)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
@Hailong-am Hailong-am merged commit 96a9046 into 2.x Sep 20, 2024
15 checks passed
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.

1 participant