Summary: | [Bugzilla] Add a checkbox to add self to the CC list when creating or editing an attachment | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> | ||||||||
Component: | Tools / Tests | Assignee: | David Kilzer (:ddkilzer) <ddkilzer> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | bfulgham, dbates, ddkilzer, lforschler, matthew_hanson, mcatanzaro, ossy, rniwa, thorton | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Csaba Osztrogonác
2013-11-08 08:47:14 PST
Created attachment 216405 [details]
Patch
WIP patch, code-review.js change is still needed to let the reviewer edit the addselfcc checkbox. (Otherwise if you review a patch, you will be added to the cc list unconditionally.)
cc reviewers touched bugzilla code Unfortunately I can't test the patch without local bugzilla setup, but I hope I won't break anything. I was bit by this twice in a recent week, I think it would be nice to have this fixed. Don't know how to test on a local bugzilla though, sadly. *** Bug 152854 has been marked as a duplicate of this bug. *** Bugzilla is already upgraded to 5.0.3: https://lists.webkit.org/pipermail/webkit-dev/2017-March/028882.html The "Details" page already has the addselfcc feature: <input type="checkbox" id="addselfcc" name="addselfcc" checked="checked"> <label for="addselfcc">Add me to CC list</label> We should add the same feature to the "Review patch" page too. (In reply to Csaba Osztrogonác from comment #0) > After a little bit debugging I found it is already fixed in bugzilla, > but a little bit later release (4.4) > > bug report: > https://bugzilla.mozilla.org/show_bug.cgi?id=138546 > Need email CC addition option on create attachment page > > Fix: > http://bzr.mozilla.org/bugzilla/trunk/revision/8268 https://github.com/bugzilla/bugzilla/commit/5b3ea4d476084d47c99e3d2e9d409fea84c16831 (In reply to Csaba Osztrogonác from comment #0) > I found an annoying bug in our bugzilla. > > When I'm not on the cc list of a bug and I review a patch in it, > the bugzilla don't cc myself to the bug. It is not good at all, > because I won't get any email from this bug if the author fixed > what I asked or something wrong happened after landing, ... I hope making a review using the review form will also auto-CC. I have missed many comments on my reviews due to this issue. Created attachment 305123 [details]
Patch adapted to Bugzilla 5.0.3
Tested on our internal test instance, and this works just as expected.
Created attachment 305154 [details] Patch with fixed paths Same as Attachment 305123 [details], but fixed the paths to apply to the full source tree. Comment on attachment 305154 [details] Patch with fixed paths View in context: https://bugs.webkit.org/attachment.cgi?id=305154&action=review > Websites/bugs.webkit.org/ChangeLog:12 > + adds the reviewed to the CC list whether the "Preview" reviewer? Committed r214401: <http://trac.webkit.org/changeset/214401> |