tree 30e3b28f4e971b526c02c9bcb6153004eea1102f
parent 797596b88111dcb3282d20936f3abba34c9c8aac
author Kamil Musin <kamilm@google.com> 1729854972 +0200
committer Paladox none <thomasmulhall410@yahoo.com> 1729860103 +0000

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
(cherry picked from commit f42012d837c1e15e0a598f41b66c999bae6ecb67)