111798 2013-03-07 17:21:44 -0800 Warning about the user not having edit rights on Bugzilla is confusing new contributors 2013-03-08 14:38:47 -0800 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 rniwa rniwa abarth alancutter benjamin dpranke eric oldest_to_newest 850687 0 rniwa 2013-03-07 17:21:44 -0800 Apparently the warning that comes out of failing to assign a bug to the user is more confusing than is helpful. A contributor who hasn't yet to become a committer doesn't really know much about how WebKit Bugzilla works and a scary warning like this one misleads the contributor to think that he needs to ask for edit rights. We should just get rid of this warning message. I've met at least 3 new contributors who have complained about this problem. 850692 1 192116 rniwa 2013-03-07 17:29:49 -0800 Created attachment 192116 Don't encourage new contributors to ask for EditBugs privileges 850710 2 dpranke 2013-03-07 17:49:58 -0800 when does this message show up? with webkit-patch upload? Can we query bugzilla and figure out if the user has editbugs permissions? 850720 3 benjamin 2013-03-07 18:06:31 -0800 (In reply to comment #2) > when does this message show up? with webkit-patch upload? > > Can we query bugzilla and figure out if the user has editbugs permissions? New users are incredibly confused by that message. 850721 4 dpranke 2013-03-07 18:10:13 -0800 (In reply to comment #3) > (In reply to comment #2) > > when does this message show up? with webkit-patch upload? > > > > Can we query bugzilla and figure out if the user has editbugs permissions? > > New users are incredibly confused by that message. new users of what? 850722 5 eric 2013-03-07 18:10:28 -0800 (In reply to comment #2) > Can we query bugzilla and figure out if the user has editbugs permissions? We could parse their permissions page, yes. I'm not sure it's worth the effort. 850774 6 benjamin 2013-03-07 19:29:12 -0800 > > New users are incredibly confused by that message. > > new users of what? Of webkit-patch. People who get started on WebKit always ask on IRC about that error. 851443 7 eric 2013-03-08 12:44:49 -0800 Honestly, I don't really care. :) We could do something more elegant. I guess I don't notice folks asking for EditBugs. I definitely notice folks who fail to ever ask for it. :) Another solution would be to just get rid of edit bugs. 851515 8 192116 rniwa 2013-03-08 14:38:43 -0800 Comment on attachment 192116 Don't encourage new contributors to ask for EditBugs privileges Clearing flags on attachment: 192116 Committed r145270: <http://trac.webkit.org/changeset/145270> 851516 9 rniwa 2013-03-08 14:38:47 -0800 All reviewed patches have been landed. Closing bug. 192116 2013-03-07 17:29:49 -0800 2013-03-08 14:38:43 -0800 Don't encourage new contributors to ask for EditBugs privileges bug-111798-20130307172553.patch text/plain 1607 rniwa SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl dmlzaW9uIDE0NTE2MikKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz ICsxLDE0IEBACisyMDEzLTAzLTA3ICBSeW9zdWtlIE5pd2EgIDxybml3YUB3ZWJraXQub3JnPgor CisgICAgICAgIFdhcm5pbmcgYWJvdXQgdGhlIHVzZXIgbm90IGhhdmluZyBlZGl0IHJpZ2h0cyBv biBCdWd6aWxsYSBpcyBjb25mdXNpbmcgbmV3IGNvbnRyaWJ1dG9ycworICAgICAgICBodHRwczov L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTExNzk4CisKKyAgICAgICAgUmV2aWV3 ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgRG9uJ3Qgc2NhcmUgcGVvcGxlIGJ5IHRo ZSB3YXJuaW5nLiBJbnN0ZWFkLCB0ZWxsIHRoZW0gdG8gaWdub3JlIGl0IGlmIHRoZXkgZG9uJ3Qg aGF2ZSBFZGl0QnVncyBwcml2aWxlZ2VzLgorCisgICAgICAgICogU2NyaXB0cy93ZWJraXRweS9j b21tb24vbmV0L2J1Z3ppbGxhL2J1Z3ppbGxhLnB5OgorCiAyMDEzLTAzLTA3ICBDaHJpc3RpYW4g Qmllc2luZ2VyICA8Y2JpZXNpbmdlckBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgQWRkIG15c2Vs ZiB0byByZXZpZXdlcnMucHkKSW5kZXg6IFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL25l dC9idWd6aWxsYS9idWd6aWxsYS5weQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9TY3JpcHRzL3dlYmtp dHB5L2NvbW1vbi9uZXQvYnVnemlsbGEvYnVnemlsbGEucHkJKHJldmlzaW9uIDE0NTAyMSkKKysr IFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvY29tbW9uL25ldC9idWd6aWxsYS9idWd6aWxsYS5weQko d29ya2luZyBjb3B5KQpAQCAtODExLDExICs4MTEsNyBAQCBjbGFzcyBCdWd6aWxsYShvYmplY3Qp OgogCiAgICAgICAgIGlmIG5vdCBzZWxmLl9oYXNfY29udHJvbChzZWxmLmJyb3dzZXIsICJhc3Np Z25lZF90byIpOgogICAgICAgICAgICAgX2xvZy53YXJuaW5nKCIiIkZhaWxlZCB0byBhc3NpZ24g YnVnIHRvIHlvdSAoY2FuJ3QgZmluZCBhc3NpZ25lZF90bykgY29udHJvbC4KLURvIHlvdSBoYXZl IEVkaXRCdWdzIHByaXZpbGVnZXMgYXQgYnVncy53ZWJraXQub3JnPwotaHR0cHM6Ly9idWdzLndl YmtpdC5vcmcvdXNlcnByZWZzLmNnaT90YWI9cGVybWlzc2lvbnMKLQotSWYgbm90LCB5b3Ugc2hv dWxkIGVtYWlsIHdlYmtpdC1jb21taXR0ZXJzQGxpc3RzLndlYmtpdC5vcmcgb3IgYXNrIGluICN3 ZWJraXQKLWZvciBzb21lb25lIHRvIGFkZCBFZGl0QnVncyB0byB5b3VyIGJ1Z3Mud2Via2l0Lm9y ZyBhY2NvdW50LiIiIikKK0lnbm9yZSB0aGlzIG1lc3NhZ2UgaWYgeW91IGRvbid0IGhhdmUgRWRp dEJ1Z3MgcHJpdmlsZWdlcyAoaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvdXNlcnByZWZzLmNnaT90 YWI9cGVybWlzc2lvbnMpIiIiKQogICAgICAgICAgICAgcmV0dXJuCiAKICAgICAgICAgaWYgY29t bWVudF90ZXh0Ogo=