search

cryptag / backchannel

Secure collaboration for activists, journalists, and you
https://www.cryptag.org/
Other
4 stars 2 forks source link

UI allows deleting messages first steps for #16 #19

Closed jimmcgaw closed 7 years ago

jimmcgaw commented 7 years ago

This doesn't actually address the underlying issue. If I understand it correctly, we want there to be some visual record of "there once was a message here, but its author deleted it" in the UI. That would require some kind of soft delete, and we render that if the soft delete boolean for the associated message is true.

elimisteve commented 7 years ago

If I understand it correctly, we want there to be some visual record of "there once was a message here, but its author deleted it" in the UI.

Just completely deleting the message and leaving no trace is cool with me! See https://github.com/cryptag/backchannel/issues/16#issuecomment-264819889 for details.

jimmcgaw commented 7 years ago

Ah, cool. In that case, review and/or merge away.

elimisteve commented 7 years ago

Merged. Great work, @jimmcgaw!