Add section to show_bug displaying the status of associated revisions

RESOLVED FIXED

Status

()

enhancement
RESOLVED FIXED
10 months ago
7 months ago

People

(Reporter: mcote, Assigned: dkl)

Tracking

Development
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 3 obsolete attachments)

Reporter

Description

10 months ago
Similar to what we did with MozReview, let's add a section to show_bug that shows the status of any revisions associated with the bug, without any translation.  Ideally we would display the same icons with the same tooltips.  The information should be limited to the summary, revision status (e.g. "Needs Review", "Accepted", "Closed", etc.), and the names and statuses of current reviewers ("Review Requested", "Blocking Review", "Accepted", "Requested Changes", etc.).
Reporter

Comment 1

10 months ago
It was discussed at a meeting that screenscraping may (unfortunately) be the best option here due to the complexity of determining statuses via the Conduit API and due to potential changes to the underlying models.
Reporter

Comment 2

10 months ago
Eventually it would be nice to hide revision attachments from the attachments table in lieu of this new table, but we'll have to leave it for now until we determine alternatives to other types of flags (e.g. uplifts).
Assignee

Comment 3

10 months ago
Posted file GitHub Pull Request (obsolete) —
Assignee: nobody → dkl
Status: NEW → ASSIGNED
Assignee

Comment 4

8 months ago
Posted file GitHub Pull Request
Attachment #9007772 - Attachment is obsolete: true

Updated

8 months ago
Depends on: 1487422
Added:

* fa-code (\f121)
* fa-times-circle-o (\f05c)
* fa-flag (\f024)
* and a few more that can be found at https://github.com/phacility/phabricator/tree/master/src/applications/differential/xaction
Attachment #9022298 - Attachment is obsolete: true
Assignee

Comment 8

7 months ago
Merged to master.
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.