Skip to content

[feat]: Added warning UIs for testset disconnect & change#3959

Merged
mmabrouk merged 2 commits intoforntend/fix-stateless-playground-bug-fixesfrom
frontend/enhance-playground-testset-sync-confirmation
Mar 12, 2026
Merged

[feat]: Added warning UIs for testset disconnect & change#3959
mmabrouk merged 2 commits intoforntend/fix-stateless-playground-bug-fixesfrom
frontend/enhance-playground-testset-sync-confirmation

Conversation

@ashrafchowdury
Copy link
Contributor

@ashrafchowdury ashrafchowdury commented Mar 11, 2026

What's new??

Added warning modals for testset sync on the playground to make sure the user gets a proper message before disconnecting the testset and changing the testset connection.

Note: This modal only appears when we have changes to sync

Details When disconnecting the testset image

When changing the testset

image

QA

  • Go to playground
  • Connect a testset
  • Make changes to the testcases on the playground
  • Try to disconnect the testset, now you should be able to see the warning modal
  • Try changing the testset, now you should be able to see the warning modal
  • Make sure all the buttons work as expected on the modals

@vercel
Copy link

vercel bot commented Mar 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agenta-documentation Ready Ready Preview, Comment Mar 11, 2026 3:39pm

Request Review

@ashrafchowdury ashrafchowdury marked this pull request as ready for review March 11, 2026 15:46
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 11, 2026
@dosubot dosubot bot added the UI label Mar 11, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 11, 2026

Railway Preview Environment

Status Destroyed (PR closed)

Updated at 2026-03-12T10:18:43.670Z

@mmabrouk mmabrouk merged commit 70a6c9d into forntend/fix-stateless-playground-bug-fixes Mar 12, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files. UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants