Closed Bug 1440828 Opened 3 years ago Closed 3 years ago

Phabricator review requests should show up on the BMO dashboard

Categories

(bugzilla.mozilla.org :: MyDashboard, defect)

Production
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: botond, Assigned: glob)

References

Details

(Keywords: conduit-triaged)

Attachments

(1 file)

pr
45 bytes, text/x-github-pull-request
Details | Review
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: nobody → glob
Component: Phabricator Extensions → Extensions: MyDashboard
Product: Conduit → bugzilla.mozilla.org
Version: unspecified → Production
Whiteboard: [phabricator-backlog]
Blocks: 1447568
Attached file pr
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
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Blocks: 1457018
No longer blocks: 1457018
Blocks: 1482110
You need to log in before you can comment on or make changes to this bug.