Skip to content

Conversation

raunakab
Copy link
Contributor

@raunakab raunakab commented Apr 21, 2025

Description

This PR adds the assistant_name to the "Chat Sessions Details" page inside of the "Query History" section.

Addresses: https://linear.app/danswer/issue/DAN-1796/add-assistant-into-query-history-display.

How Has This Been Tested?

UI change; visually tested and confirmed to work.

@raunakab raunakab requested a review from a team as a code owner April 21, 2025 12:19
Copy link

vercel bot commented Apr 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
internal-search ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 21, 2025 0:26am

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Added assistant name display to the Chat Sessions Details page in Query History, enhancing session information visibility and traceability.

  • Added new Text component in /web/src/app/ee/admin/performance/query-history/[id]/page.tsx to display assistant_name from chatSessionSnapshot
  • Implementation maintains existing styling patterns with flex and whitespace-normal classes for consistent layout

1 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

@rkuo-danswer rkuo-danswer disabled auto-merge April 21, 2025 17:26
@raunakab raunakab merged commit e3218d3 into main Apr 22, 2025
8 of 11 checks passed
@raunakab raunakab deleted the feat/add-assistant-to-query-history branch April 22, 2025 16:35
aronszanto pushed a commit to aronszanto/onyx that referenced this pull request Apr 26, 2025
* Add assistant name to UI

* Fix tailwind styling class
AnkitTukatek pushed a commit to TukaTek/onyx that referenced this pull request Sep 23, 2025
* Add assistant name to UI

* Fix tailwind styling class
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants