tree 40fa2f0ea0b6e36647c307c1913a916c34b341ae
parent ac4c1888e9cb5ad51dffc0e5cd58ca067f189b18
author Kamil Musin <kamilm@google.com> 1729854972 +0200
committer Kamil Musin <kamilm@google.com> 1729858407 +0200

Update attention set behaviour when commenting on a thread.

Previously the rule worked like this:
When reply comment is added on a thread, every previous commenters
(except for the user leaving a reply) on the thread is added if:
- They are not the user leaving a reply
- If thread is unresolved
- If thread is resolved, and the commenter have not voted Code-Review
  with a maximum vote yet.

After this change:
- If change owner replies and thread is unresolved: every commenter on
  the thread is brought back to attention set
- If change owner replies and thread is resolved: only commenters that
  haven't voted Code-Review are brought to attention set
- If reviewer replies and thread is unresolved, only bring owner to
  attention set
- If reviewer replies and thread is resolved, bring all other commenters
  who haven't voted Code-Review to attention set.

The logic doesn't account for users who don't have permissions to leave
a max CR vote. But this is the same for the current state as well. This
can be a further improvement in the future.

This is FE-only change, this will affect only comments published via
Web-UI. There will be a corresponding check to the BE logic in a
subsequent change.

Release-Notes: Attention set when commenting on a thread takes into
  account owner and unresolved status.
Google-Bug-Id: b/347949247
Change-Id: I849a8dda069c74d97c75ce86706f35ea64717670