Skip to content

fix: BUG-FIX-11704-Dev #1563

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

Closed
wants to merge 4 commits into from
Closed

Conversation

UtkarshMishra-Microsoft
Copy link
Contributor

@UtkarshMishra-Microsoft UtkarshMishra-Microsoft commented Dec 18, 2024

Purpose

Deploy CWYD, user should have multiple chat threads (chat history)
Refresh the page and in new section say "Hi", after user gets response, ask second question which will take at least 30sec to generate the complete response. Ex: show me a list of contracts from the state of Alabama that were published 2022 to 2024 OR What are the reporting requirements for the vendor under this agreement?

Actual: When response is being generated, try to move to other chat histories. Look history from first thread where currently response is getting generated, will appear on each chat history thread from past. Once response is generated completely, then we can see previous chat history properly.

Expected: When response is getting generated, User should not be able to move other sections

  • ...

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

Roopan-Microsoft and others added 4 commits December 3, 2024 04:33
Co-authored-by: Ross Smith <ross-p-smith@users.noreply.github.com>
…1512)

Co-authored-by: Francia Riesco <friesco@microsoft.com>
Co-authored-by: Pavan Kumar <v-kupavan.microsoft.com>
Co-authored-by: Pavan-Microsoft <v-kupavan@microsoft.com>
Co-authored-by: Francia Riesco <Fr4nc3@users.noreply.github.com>
Co-authored-by: Prajwal D C <v-dcprajwal@microsoft.com>
Copy link

Coverage

Coverage Report •
FileStmtsMissCoverMissing
TOTAL364494574% 
report-only-changed-files is enabled. No files were changed during this commit :)

Tests Skipped Failures Errors Time
370 0 💤 0 ❌ 0 🔥 53.964s ⏱️

@UtkarshMishra-Microsoft UtkarshMishra-Microsoft changed the base branch from main to dev December 18, 2024 10:59
@UtkarshMishra-Microsoft UtkarshMishra-Microsoft changed the base branch from dev to main December 18, 2024 11:07
@UtkarshMishra-Microsoft UtkarshMishra-Microsoft changed the base branch from main to dev December 18, 2024 11:09
@UtkarshMishra-Microsoft UtkarshMishra-Microsoft changed the base branch from dev to main December 18, 2024 12:24
@UtkarshMishra-Microsoft UtkarshMishra-Microsoft changed the base branch from main to dev December 18, 2024 12:37
@UtkarshMishra-Microsoft UtkarshMishra-Microsoft deleted the BUG-FIX-11704-Dev branch December 24, 2024 09:18
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.

3 participants