Closed
Bug 1440828
Opened 6 years ago
Closed 6 years ago
Phabricator review requests should show up on the BMO dashboard
Categories
(bugzilla.mozilla.org :: MyDashboard, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: botond, Assigned: glob)
References
Details
(Keywords: conduit-triaged)
Attachments
(1 file)
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•6 years ago
|
||
I'll have to think about this, but it's an interesting idea.
Keywords: conduit-triaged
Whiteboard: [phabricator-backlog]
Comment 2•6 years ago
|
||
This is one of the biggest drags when using Phabricator for me, fwiw.
Component: Phabricator Extensions → Extensions: MyDashboard
Product: Conduit → bugzilla.mozilla.org
Version: unspecified → Production
Comment 4•6 years 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•6 years 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•6 years ago
|
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•