-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature/#87361 delete all conversations in history #80
Conversation
…I with delete all functionality
…nhance Redux integration
…sOptions component
…eviousConversationsOptions component
… in webchat settings and components
… conversations state
…iousConversationsOptions
… in WebchatUI component
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This comment was marked as resolved.
This comment was marked as resolved.
When I click the 'delete all conversations' button, I do not see the list of previous conversations under the modal. |
Fixed it |
Actually the path was using a default fill I have set the path color now |
I have a single conversation. I delete it. Then I go to the Home screen and click "Start conversation". The conversation continues (the history is not erased). I would expect it to be empty. You probably want to make sure the session is new in this case. |
The button in the modal is not auto focused during keyboard navigation when re-opened. It does focus the first time though. |
Just to confirm this is only when toggling delete all conversations dialog right ? coz the other modal seems fine |
…disconnect and switchSession actions
@kwinto I have fixed this one . Can you please check again .
|
…onfirm delete action
…agement and close handling
src/webchat-ui/components/presentational/previous-conversations/DeleteAllConversations.tsx
Outdated
Show resolved
Hide resolved
src/webchat-ui/components/presentational/previous-conversations/helpers.ts
Outdated
Show resolved
Hide resolved
src/webchat/store/previous-conversations/previous-conversations-reducer.ts
Show resolved
Hide resolved
…mproved state handling and responsiveness
…include channel in storage key handling
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
Success criteria
How to test
Delete all Conversations
Delete conversation
Security
Additional considerations
Documentation Considerations
Ensure documentation includes details on the new "Delete all conversations" button, user confirmation alerts, and compliance with accessibility guidelines.