Danny Liu

Results 18 comments of Danny Liu

> @dannyl1u Thanks for the hard work, I'll merge this for now! Delete confirmation feature sounds like it would be nice to have, feel free to make a separate PR....

+1 I agree this would be a useful feature to keep a clean conversation history and avoid having side questions, clarifications, etc. affect the context. One solution would be to...

Sounds good, @tjbck is it okay if I push my changes to the `delete-message` branch or should I make a separate branch for the PR?

@tjbck published my PR, please review when you get the chance, thank you!

@taidbui thanks for letting me know, will log that bug in a separate issue!

Closing this feature request as we have decided to not delete all subsequent messages, only the selected prompt/response pair. Thus, a confirmation is not needed at this time

@tjbck please verify that removing this line won't cause any vulnerabilities or issues with XSS etc..