Skip to content

Feat/assistant scroll #31042

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

Merged
merged 3 commits into from
Dec 10, 2024
Merged

Feat/assistant scroll #31042

merged 3 commits into from
Dec 10, 2024

Conversation

SaxonF
Copy link
Contributor

@SaxonF SaxonF commented Dec 10, 2024

What kind of change does this PR introduce?

This is a bit of a hodge hodge of a PR but it does a few things:

  • Updates scroll behavior so that you can scroll up in chat whilst a new response is being streamed. The useAutoScroll hook is borrowed from database.build
  • Adjusts prompt to ask for confirmation when about to write a destructive query
  • Adjusts styling on confirmation to be a little more in your face
scroll-and-confirmation.mp4

@SaxonF SaxonF requested a review from a team as a code owner December 10, 2024 10:33
Copy link

vercel bot commented Dec 10, 2024

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

Name Status Preview Comments Updated (UTC)
studio-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 10, 2024 10:33am
5 Skipped Deployments
Name Status Preview Comments Updated (UTC)
design-system ⬜️ Ignored (Inspect) Dec 10, 2024 10:33am
studio ⬜️ Ignored (Inspect) Dec 10, 2024 10:33am
studio-self-hosted ⬜️ Ignored (Inspect) Dec 10, 2024 10:33am
docs ⬜️ Skipped (Inspect) Dec 10, 2024 10:33am
zone-www-dot-com ⬜️ Skipped (Inspect) Dec 10, 2024 10:33am

Copy link

supabase bot commented Dec 10, 2024

This pull request has been ignored for the connected project xguihxuzqibwxjnimxev because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@SaxonF SaxonF merged commit 9f718e9 into master Dec 10, 2024
15 checks passed
@SaxonF SaxonF deleted the feat/assistant-scroll branch December 10, 2024 21:00
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