Phabricator review requests should show up on the BMO dashboard

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
Last year
10 months ago

People

(Reporter: botond, Assigned: glob)

Tracking

({conduit-triaged})

Production
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

pr
45 bytes, text/x-github-pull-request
Details | Review
Reporter

Description

Last year
Unlike Splinter and MozReview review requests, Phabricator review requests do not show up on the BMO dashboard, because Phabricator does not set r? flags in Bugzilla.

I'm told that having Phabricator set r? flags in Bugzilla has technical challenges. I would be happy with a solution that allowed Phabricator review requests to show up on the BMO dashboard without setting r? flags (e.g. via a BMO extension that adds a new section to the dashboard).

However, I think that no longer having a single place to look at all your requests (needinfo, review, etc.) would be an unfortunate productivity regression.
I'll have to think about this, but it's an interesting idea.
Keywords: conduit-triaged
Whiteboard: [phabricator-backlog]
This is one of the biggest drags when using Phabricator for me, fwiw.
Assignee

Updated

Last year
Assignee: nobody → glob
Assignee

Updated

Last year
Component: Phabricator Extensions → Extensions: MyDashboard
Product: Conduit → bugzilla.mozilla.org
Version: unspecified → Production
Assignee

Updated

Last year
Whiteboard: [phabricator-backlog]
Assignee

Updated

Last year
Blocks: 1447568
Posted file pr

Comment 4

Last year
Mark's post on m.d.platform talks about "my dashboard" and "notifications indicator". Will this patch also cause the review requests to show up in the request.cgi pages, or should I file a separate bug for that? It's not really clear from the discussion here.
(In reply to :Gijs from comment #4)
> Mark's post on m.d.platform talks about "my dashboard" and "notifications
> indicator". Will this patch also cause the review requests to show up in the
> request.cgi pages, or should I file a separate bug for that? It's not really
> clear from the discussion here.

As coded in the PR for this bug, no. But if you would like to file a separate bug we can look 
at extending requests.cgi to also show the phabricator requests in its own table. It can just
reuse the same code from the PR.

dkl

Updated

Last year
Blocks: 1450996
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Blocks: 1457018
No longer blocks: 1457018

Updated

10 months ago
Blocks: 1482110
You need to log in before you can comment on or make changes to this bug.