Comment #3 on issue 3013 by Deleting a file attachment does not delete comments about it from issue summary table

I can see an argument for that, but there's another argument that I believe requires that we keep behavior as-is.

Say I upload a screenshot, and people make comments on it. I may, as part of a subsequent update, replace that screenshot with a newer version, without addressing their concerns. For example, maybe I tweaked the look of something, but I didn't fix a typo they found.

By altering this behavior to erase those issues from the removed screenshot, I lose very valuable feedback that may still be very relevant. I don't want to do that. Just because a file attachment (or diff) was replaced, doesn't mean the previous feedback has been addressed.

What *should* be done is that the owner of the review request should close the issues once they are fixed. Reviews should *never* disappear. It's up to the owner to respond to any open issues, and having the system auto-close them is asking for trouble.

You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:

You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
To post to this group, send email to
Visit this group at
For more options, visit

Reply via email to