fix: display conversation title in delete confirmation modal#11818
fix: display conversation title in delete confirmation modal#11818neubig merged 7 commits intoOpenHands:mainfrom
Conversation
|
amanape will be available next week. I'll get him to take a look. Thank you! |
|
Hi @naaa760, according to the image below, we can display the conversation title directly within the confirmation message instead of presenting it on a separate line. 
For example, the confirmation message could be structured as follows.
|
|
@hieptl |
42eb75c to
3b135be
Compare
hieptl
left a comment
There was a problem hiding this comment.
Hi @naaa760, I have verified the ticket locally, and the feature is working as expected. However, before approving and merging this pull request, could you please add tests for your changes? We are aiming to enforce stronger testing practices on the frontend. Thank you! 🙏
77f2fe8 to
b0a0d3b
Compare
|
Hello @naaa760, Could you please let me know if you are currently working on this pull request? Thank you! 🙏 |
amanape
left a comment
There was a problem hiding this comment.
I have simplified the test, and made a small style change to highlight the conversation title:
Summary of PR
Displays the conversation title in the delete confirmation modal so users can verify which conversation they're deleting.
Change Type
Checklist
Fixes
Resolves #11809