[DERCBOT-1755] KPI Satisfaction - Feedback User#1963
Merged
vsct-jburet merged 1 commit intotheopenconversationkit:masterfrom Jan 13, 2026
Merged
Conversation
0d34103 to
c0ffbea
Compare
vsct-jburet
approved these changes
Jan 5, 2026
Contributor
|
@assouktim I add copilot review. You can close all useless comments ;) |
There was a problem hiding this comment.
Pull request overview
This PR implements a user feedback feature (KPI Satisfaction tracking) that allows users to provide thumbs up/down feedback on bot actions. The feature tracks feedback votes (UP/DOWN) in action metadata, persists them in the database, and displays them in the admin UI metrics dashboard.
Key Changes:
- Added
ActionFeedbackandFeedbackEventclasses to handle user feedback - Extended
ActionMetadatawith feedback field and created corresponding MongoDB mappings - Implemented feedback counting and filtering in
UserTimelineMongoDAO - Updated web connector to support feedback submission via
FeedbackParams - Enhanced admin UI to display feedback metrics and icons in dialog logs
Reviewed changes
Copilot reviewed 20 out of 21 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
bot/engine/src/main/kotlin/engine/action/ActionFeedback.kt |
New data class defining feedback vote types (UP/DOWN) with timestamp |
bot/engine/src/main/kotlin/engine/event/FeedbackEvent.kt |
New event type for handling feedback submissions |
bot/engine/src/main/kotlin/engine/action/ActionMetadata.kt |
Added feedback field to store action feedback |
bot/engine/src/main/kotlin/definition/EventListenerBase.kt |
Implemented feedback event handling to persist feedback to user timeline |
bot/storage-mongo/src/main/kotlin/UserTimelineMongoDAO.kt |
Added feedback filtering and counting queries for metrics |
bot/connector-web/src/main/kotlin/WebConnectorRequest.kt |
Added feedback parameter handling to convert web requests to FeedbackEvents |
bot/connector-web/src/main/kotlin/WebMessage.kt |
Added actionId field to support feedback linking |
bot/connector-web/src/main/kotlin/WebMessageProcessor.kt |
Updated message processor to include actionId in responses |
bot/connector-web-model/src/main/kotlin/ai/tock/bot/connector/web/WebConnectorRequest.kt |
Added FeedbackParams model and interface support |
bot/admin/web/src/app/shared/model/dialog-data.ts |
Added TypeScript interfaces for feedback data structures |
bot/admin/web/src/app/metrics/metrics-board/metrics-board.component.ts |
Added feedback count aggregation logic |
bot/admin/web/src/app/metrics/metrics-board/metrics-board.component.html |
Added UI display for feedback metrics (thumbs up/down counts) |
bot/admin/web/src/app/shared/components/chat-ui/chat-ui-dialog-logger/chat-ui-dialog-logger.component.html |
Added feedback icons display in dialog message sender names |
bot/admin/server/src/main/kotlin/model/DialogsSearchQuery.kt |
Added feedback filter parameter to dialog search |
bot/admin/server/src/main/kotlin/BotAdminService.kt |
Added feedback data grouping for metrics response |
bot/engine/src/main/kotlin/admin/dialog/DialogReportQuery.kt |
Added feedback filter to dialog query model |
bot/engine/src/main/kotlin/admin/dialog/DialogStatsQueryResult.kt |
Added feedback count fields to stats result |
bot/storage-mongo/target/generated-sources/kapt/compile/ai/tock/bot/engine/action/ActionMetadata_.kt |
Generated Kotlin property path mappings for feedback field |
bot/connector-web-model/src/main/kotlin/ai/tock/bot/connector/web/send/WebMessageContent.kt |
Added actionId to message contract |
bot/engine/src/main/kotlin/engine/TockConnectorController.kt |
Removed empty line (formatting fix) |
bot/connector-web/src/main/kotlin/WebConnectorCallback.kt |
Overridden eventSkipped to send response for feedback events |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
bot/connector-web-model/src/main/kotlin/ai/tock/bot/connector/web/WebConnectorRequest.kt
Show resolved
Hide resolved
...src/app/shared/components/chat-ui/chat-ui-dialog-logger/chat-ui-dialog-logger.component.html
Outdated
Show resolved
Hide resolved
[DERCBOT-1752] Add feedback dialog filter
c0ffbea to
9ba365f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.