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

fix: Retain Chat Session When Switching from Chat to Document Tab #200

Merged
merged 4 commits into from
Feb 21, 2025

Conversation

Prasanjeet-Microsoft
Copy link
Contributor

Purpose

  • This PR addresses the issue where the chat session is cleared when switching from the 'Chat' tab to the 'Document' tab in document's details section. The fix ensures that the chat session persists even after switching between the two tabs

Does this introduce a breaking change?

  • Yes
  • [ x] No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  1. Functionality:

    • Confirm that the chat session is preserved after switching from 'Chat' to 'Document' and back to 'Chat'.
    • Ensure that no data loss or session clearing happens when navigating between these tabs.
  2. Testing:

    • Reproduce the issue described in the bug report to verify it is resolved.
    • Check if the bug fix introduces any new issues related to tab navigation or chat sessions.
  3. UI/UX:

    • Ensure the user experience remains smooth, with no disruptions in the chat when switching tabs.
    • Check for any UI inconsistencies or regressions related to the chat display.

Roopan-Microsoft and others added 4 commits February 17, 2025 19:25
fix: Merging Dev fixes to main branch
Updated Deployment Trigger Schedule
chore: Updated workflow names and trigger schedule
@Roopan-Microsoft Roopan-Microsoft merged commit d154800 into dev Feb 21, 2025
5 checks passed
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.

4 participants