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

Studio: Add UI for clearing chat messages #250

Merged
merged 6 commits into from
Jun 14, 2024

Conversation

katinthehatsite
Copy link
Contributor

Closes https://github.com/Automattic/dotcom-forge/issues/7697
Related to https://github.com/Automattic/dotcom-forge/issues/7411

Proposed Changes

This PR adds a dropdown to clear chat messages.

Testing Instructions

  • Pull the changes from this branch
  • Start the app with the AI assistant STUDIO_AI=true npm start
  • Navigate to the assistant tab and start conversation with the AI assistant
  • Once there are a couple of messages, click on the three dots by the input field
  • Observe the dropdown to Clear chat history
  • Observe the popup that asks for confirmation to delete the chat (not sure if this is completely necessary to have?)
  • Confirm the deletion of the chat history
  • Observe that the messages are removed and you are back to seeing the welcome messages and prompts
Capture d’écran, le 2024-06-14 à 14 12 34

The was already the PR that attempted this change before so I reused some of the code from https://github.com/Automattic/studio/pull/206/files

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@katinthehatsite katinthehatsite requested a review from a team June 14, 2024 13:37
Copy link
Contributor

@jeroenpf jeroenpf left a comment

Choose a reason for hiding this comment

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

Nice work 👍 LGTM!

@katinthehatsite katinthehatsite merged commit 8513d6f into trunk Jun 14, 2024
11 checks passed
@katinthehatsite katinthehatsite deleted the add/option-to-clear-input branch June 14, 2024 14:23
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.

None yet

2 participants