Closed
Bug 1469378
Opened 7 years ago
Closed 7 years ago
Update feed daemon to only manage subscribers on a revision if the bug is private, otherwise leave it alone
Categories
(bugzilla.mozilla.org :: Phabricator Integration, enhancement)
Tracking
()
RESOLVED
FIXED
People
(Reporter: dkl, Assigned: dkl)
References
Details
(Keywords: conduit-triaged)
Attachments
(1 file)
Currently we only show a warning at the top of the revision when the revision is private and it states that you must use Bugzilla to manage the subscribers list. This is also true for public revisions that have bug ids attached and so we need to show some sort of warning anytime a bug id is attached.
|
||
Comment 1•7 years ago
|
||
(In reply to David Lawrence [:dkl] from comment #0)
> Currently we only show a warning at the top of the revision when the
> revision is private and it states that you must use Bugzilla to manage the
> subscribers list. This is also true for public revisions that have bug ids
> attached and so we need to show some sort of warning anytime a bug id is
> attached.
I was under the impression we would not manage the subscribers of public bugs? :mcote have you changed your mind on this?
Flags: needinfo?(mcote)
|
||
Comment 2•7 years ago
|
||
I hadn't; I think this was just a misunderstanding of requirements. Admittedly I don't particularly like that we treat the subscribers list differently in the two situations. However, going to the bug to add CCs for public patches seems to be even worse UX than the inconsistent behaviour.
We should revert this, at least for now. dkl, agreed?
Flags: needinfo?(mcote) → needinfo?(dkl)
|
Assignee | |
Comment 4•7 years ago
|
||
(In reply to Mark Côté [:mcote] from comment #2)
> I hadn't; I think this was just a misunderstanding of requirements.
> Admittedly I don't particularly like that we treat the subscribers list
> differently in the two situations. However, going to the bug to add CCs for
> public patches seems to be even worse UX than the inconsistent behaviour.
>
> We should revert this, at least for now. dkl, agreed?
Sure thing. Shouldn't be that difficult. Only sync cc's when bug is private and not when public. Nothing to be done on the Phab side as the behavior already matches the desired outcome.
dkl
Assignee: nobody → dkl
Status: NEW → ASSIGNED
Flags: needinfo?(dkl)
|
|
Assignee | |
Updated•7 years ago
|
Summary: Post note at top of revision when linked to bug id that subscribers are synced with bugs cc list only → Update feed daemon to only manage subscribers on a revision if the bug is private, otherwise leave it alone
|
|
||
Updated•7 years ago
|
Keywords: conduit-triaged
|
|
Assignee | |
Comment 5•7 years ago
|
||
Component: Phabricator → Extensions: PhabBugz
Product: Conduit → bugzilla.mozilla.org
Version: unspecified → Production
|
|
Assignee | |
Comment 6•7 years ago
|
||
Merged to master.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•