The feed daemon loops over current flags and if the setter of the current flag is the same as the new flag being set, it skips adding that flag. Even if the current flags status is '?'. Originally it was designed this way thinking that only Phabricator would be setting review flags for those phabricator revision related patches (those that redirect to phabricator revisions). The change will only skip if the review flag is set to '+'. Otherwise it will add another review+ flag separately. This may have the side effect of having more than one review flag set for an attachment. One with the status '?' set manually by a user, and one with status '+' set by phab-bot. dkl
Status: ASSIGNED → RESOLVED
Last Resolved: 11 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.