> :imadueme, is there any chance you could investigate what's going on here? My current theory is that Phabricator doesn't treat "Request Changes" and then "Approve" as removing the need for changes, even when it's from the same user.
I've confirmed that this is not the case and this work as they should on phabricator-dev.
Bug 1520981 Comment 5 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
> :imadueme, is there any chance you could investigate what's going on here? My current theory is that Phabricator doesn't treat "Request Changes" and then "Approve" as removing the need for changes, even when it's from the same user.
I've confirmed that this is not the case and things work as they should on phabricator-dev.