1321592 - Update Bugzilla Etiquette and add Abuse Policy

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: General, defect, P1)

Description

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Update the Etiquette page, add an abuse policy page, and link to abuse policy from the front page.

Comment 1

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Opened https://github.com/mozilla-bteam/bmo/pull/32

Comment 2

[Dylan Hardison [:dylan] (he/him)]

Attachment: autolander isn't doing its job

Comment 3

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Attachment: Updated Patch

Updated patch, resolving conflict with etiquette.html.tmpl

Comment 4

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Attachment: Updated Patch (diff so there's just one set of changes)

Updated diff that resolves the conflict in etiquette.html.tmpl.

Comment 5

[David Lawrence [:dkl]]

Comment on attachment 8818389 [details] [diff] [review]
Updated Patch (diff so there's just one set of changes)

Review of attachment 8818389 [details] [diff] [review]:
-----------------------------------------------------------------

t/009bugwords.t ........... 1/734
#   Failed test 'extensions/BMO/template/en/default/pages/etiquette.html.tmpl contains invalid bare words (e.g. 'bug') --WARNING'
#
#   at t/009bugwords.t line 68.
#
t/009bugwords.t ........... 687/734 # Looks like you failed 1 test of 734.
t/009bugwords.t ........... Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/734 subtests

Look for any words such as Bugzilla, Bugs, bugs, bug, etc. and replace with the equivalent [% terms.XXX %]. You can ignore anything inside of html tags or [% %].
You can check yourself using 'perl t/009bugwords.t'.

Lots of trailing whitespace in various files.

Also rename extensions/BMO/template/en/default/pages/anti-abuse.html to extensions/BMO/template/en/default/pages/anti-abuse.html.tmpl

::: extensions/BMO/template/en/default/account/create.html.tmpl
@@ +146,5 @@
>    </li>
>    <li>
> +    Your understand that your conduct on this site is subject to both <a href="/page.cgi?id=etiquette.html">[% terms.Bugzilla %] etiquette</a>, 
> +    and <a href="https://www.mozilla.org/about/governance/policies/participation/">Mozilla's participation guidelines</a>, 
> +    and in turn are governed by our <a href="/page.cgi?id=anti-abuse.html">anti-abuse policy</a>. By creating an account, 

Nit: remove trailing whitespace. Also it is 'page.cgi...' not '/page.cgi...'. That way it will work either in production or our devel environments.

@@ +171,5 @@
> +    <td>
> +    I have read <a href="/page.cgi?id=etiquette.html">[% terms.Bugzilla %] etiquette</a>
> +    and <a href="https://www.mozilla.org/about/governance/policies/participation/">Mozilla's participation guidelines</a>
> +    and agree to abide by them, and 
> +    I acknowledge, and have read the <a href="/page.cgi?id=anti-abuse.html">anti-abuse policy</a>.

remove forward slashes for both page.cgi links.

@@ +180,4 @@
>      <td>
> +    <input type="hidden" id="token" name="token" value="[% issue_hash_token(['create_account']) FILTER html %]">
> +    <input type="submit" value="Create Account">
> +    </td>  

Nit: remove extra whitespace. Also we tab 2 spaces in side html tags so instead:

<td>
  <input type="hidden" id="token" name="token" value="[% 
    issue_hash_token(['create_account']) FILTER html %]">
  <input type="submit" value="Create Account">
</td>

Same for other changes you made.

::: extensions/BMO/template/en/default/pages/anti-abuse.html
@@ +16,5 @@
> +  # Copyright (C) 1998 Netscape Communications Corporation. All
> +  # Rights Reserved.
> +  #
> +  # Contributor(s): Emma Humphries <[e-mail]>
> +  #%]

We use a new license blurb in new files that looks like this:

[%# This Source Code Form is subject to the terms of the Mozilla Public
  # License, v. 2.0. If a copy of the MPL was not distributed with this
  # file, You can obtain one at http://mozilla.org/MPL/2.0/.
  #
  # This Source Code Form is "Incompatible With Secondary Licenses", as
  # defined by the Mozilla Public License, v. 2.0.
  #%]

@@ +21,5 @@
> +
> +[% PROCESS global/header.html.tmpl 
> +   title = "Bugzilla Anti-Abuse Policy" 
> +   style = "li { margin: 5px } .heading { font-weight: bold }" %]
> +

Nit: Add a header such as <h2>Anti-Abuse Policy</h2>, etc.

::: extensions/BMO/template/en/default/pages/etiquette.html.tmpl
@@ +26,5 @@
>     style = "li { margin: 5px } .heading { font-weight: bold }" %]
>  
>  <p>
> +It is our intention that [% terms.Bugzilla %] remains a useful tool for reporting 
> +and commenting on [% terms.bugs %], feature-requests, and tasks for the Mozilla community.

2 space indentions.

@@ +46,1 @@
>  </p>

I almost think it would look better and be fine if we put all of the sentences above together in one paragraph instead of breaking out into several paragraphs.

Comment 6

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Attachment: 1321592.etiquette.diff

Patch incorporating :dkl's comments.

Comment 7

[David Lawrence [:dkl]]

Comment on attachment 8819107 [details] [diff] [review]
1321592.etiquette.diff

I made a few fixes and alterations but overall looked good. Thanks Emma.

To https://github.com/mozilla-bteam/bmo.git
   d3a4b61..50fe01e  master -> master

dkl

Comment 8

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Thanks. Is this on bugzilla.allizom.org? I want to take a look before it goes out and I need to check to make sure MOC is ready to handle the abuse reports.

Comment 9

[David Lawrence [:dkl]]

(In reply to Emma Humphries ☕️ [:emceeaich] (UTC-8) +needinfo me from comment #8)
> Thanks. Is this on bugzilla.allizom.org? I want to take a look before it
> goes out and I need to check to make sure MOC is ready to handle the abuse
> reports.

Yes it is there. 

dkl

Comment 10

[Dylan Hardison [:dylan] (he/him)]

Note this isn't ready to go out yet, so is not resolved.

Comment 11

[David Lawrence [:dkl]]

(In reply to Dylan Hardison [:dylan] from comment #10)
> Note this isn't ready to go out yet, so is not resolved.

Backed out. We will re-push out after the beginning of the year. Just let me know. Sorry for being trigger happy.

dkl

Comment 13

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

:dkl can you deploy the pull request to dev?

Comment 14

[David Lawrence [:dkl]]

(In reply to Emma Humphries ☕️ [:emceeaich] (UTC-8) +needinfo me from comment #13)
> :dkl can you deploy the pull request to dev?

It is currently on bugzilla-dev

https://bugzilla-dev.allizom.org/page.cgi?id=anti-abuse.html

dkl

Comment 15

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Attachment: etiquette.diff

Updated diff file.

Comment 16

[Emma Humphries ☕️🎸🧞‍♀️✨ (she/they) [:emceeaich] (Pacific Time) use needinfo]

Attachment: etiquette.diff

Updated patch with new email address for contacting moderators.

Comment 17

[David Lawrence [:dkl]]

Comment on attachment 8831388 [details] [diff] [review]
etiquette.diff

r=dkl. One minor error that I will fix on commit. Good to go. Shall I commit it?

Comment 18

[David Lawrence [:dkl]]

To https://github.com/mozilla-bteam/bmo.git
   92096d7..1451ed6  development -> development


Should be on bugzilla-dev.allizom.org momentarily.

dkl

Comment 19

[David Lawrence [:dkl]]

To https://github.com/mozilla-bteam/bmo.git
   b139c33..1cb4e01  master -> master

Comment 20

[David Lawrence [:dkl]]

This is now live.

dkl