Page MenuHomePhabricator

Enable semantic relationship between code review changesets and maniphest tasks in phabricator (show "Related Gerrit Patches")
Closed, ResolvedPublic

Assigned To
Authored By
mmodell
Aug 6 2019, 2:57 PM
Tokens
"Love" token, awarded by Aklapper."Barnstar" token, awarded by Ammarpad."Like" token, awarded by fgiunchedi."100" token, awarded by MarcoAurelio."Mountain of Wealth" token, awarded by Krenair."Barnstar" token, awarded by greg."Love" token, awarded by Tgr."Yellow Medal" token, awarded by matmarex."Love" token, awarded by Dzahn."Meh!" token, awarded by zeljkofilipin."Love" token, awarded by Quiddity.

Description

It should be possible to define the type of relationship between a task and an associated review.
Additionally, that relationship, plus the status of the code review, should be displayed at the top of the Phabricator task (somewhere near the @mentions section)

Event Timeline

This would probably involve some changes in phabricator as well as the gerrit plugin: its-phabricator

mmodell triaged this task as Medium priority.Aug 6 2019, 6:18 PM

Presumably this is about Gerrit changesets?

Related:

hashar added subscribers: hashar, Krinkle.

I guess that would address a feature I have listed T209463 namely for each task add a link to a Gerrit search query for bug:Txxxx ( https://gerrit.wikimedia.org/r/#/q/bug:T12345 ) :] I have marked it as a duplicate of your task :]

10/10 totally aced.

Example https://gerrit.wikimedia.org/r/c/548964 which has Bug: T236746 and is thus attached to T236746. Under that task description:

somelevelofawesomeness.png (173×437 px, 10 KB)

As far as I concerned that fully address my idea from T209463 and even exceed it. Thank you Santa Mukunda!

I have no idea how you managed to implement and deploy that in less than four hours though.

matmarex added a subscriber: matmarex.

This is the best usability improvement on Phabricator since the day the "Close As Duplicate" button was added.

antoine-approve

(comment best seen via the web interface)

Aklapper renamed this task from Enable semantic relationship between code review changesets and maniphest tasks in phabricator to Enable semantic relationship between code review changesets and maniphest tasks in phabricator (show "Related Gerrit Patches").Dec 11 2019, 10:59 AM