Phabricator review requests should show up on the BMO dashboard

RESOLVED FIXED

Status

()

RESOLVED FIXED
9 months ago
3 months ago

People

(Reporter: botond, Assigned: glob)

Tracking

({conduit-triaged})

Production
conduit-triaged
Dependency tree / graph

Details

Attachments

(1 attachment)

(Reporter)

Description

9 months ago
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.

Comment 1

9 months ago
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

8 months ago
Assignee: nobody → glob
(Assignee)

Updated

8 months ago
Component: Phabricator Extensions → Extensions: MyDashboard
Product: Conduit → bugzilla.mozilla.org
Version: unspecified → Production
(Assignee)

Updated

8 months ago
Whiteboard: [phabricator-backlog]
(Assignee)

Updated

8 months ago
Blocks: 1447568

Comment 4

8 months ago
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.

Comment 5

8 months ago
(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

8 months ago
Blocks: 1450996
Status: NEW → RESOLVED
Last Resolved: 7 months ago
Resolution: --- → FIXED

Updated

7 months ago
Blocks: 1457018
Blocks: 1457489

Updated

7 months ago
No longer blocks: 1457018

Updated

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