launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad:please-copy-oops-ids into launchpad:master.

Commit message:
Advise copying and pasting OOPS IDs

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/380353

People often report OOPSes without including the OOPS ID, or send us a screenshot from which we end up transcribing the OOPS ID.  Try to persuade them to copy and paste the OOPS ID instead.

I'm not sure how much difference this will make, but you never know.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:please-copy-oops-ids into launchpad:master.

diff --git a/lib/lp/app/templates/launchpad-requestexpired.pt b/lib/lp/app/templates/launchpad-requestexpired.pt
index 2afe81b..df768ca 100644
--- a/lib/lp/app/templates/launchpad-requestexpired.pt
+++ b/lib/lp/app/templates/launchpad-requestexpired.pt
@@ -21,6 +21,11 @@
         Trying again in a couple of minutes might work.
       </p>
       <p>
+        If you report this as a bug, please include the error ID below,
+        preferably by copying and pasting it rather than by taking a
+        screenshot.
+      </p>
+      <p>
         (Error <abbr>ID</abbr>:
         <tal:oops replace="structure view/oops_id_text" />)
       </p>
diff --git a/lib/lp/app/templates/oops.pt b/lib/lp/app/templates/oops.pt
index 82a88da..dcb9fa6 100644
--- a/lib/lp/app/templates/oops.pt
+++ b/lib/lp/app/templates/oops.pt
@@ -32,6 +32,11 @@
           Apologies for the inconvenience.
         </p>
         <p>
+          If you report this as a bug, please include the error ID below,
+          preferably by copying and pasting it rather than by taking a
+          screenshot.
+        </p>
+        <p>
           (Error <abbr>ID</abbr>:
           <tal:oops replace="structure view/oops_id_text" />)
         </p>
diff --git a/lib/lp/services/webapp/templates/oops-veryplain.pt b/lib/lp/services/webapp/templates/oops-veryplain.pt
index 797328e..ce05b72 100644
--- a/lib/lp/services/webapp/templates/oops-veryplain.pt
+++ b/lib/lp/services/webapp/templates/oops-veryplain.pt
@@ -35,6 +35,11 @@
         Apologies for the inconvenience.
       </p>
       <p>
+        If you report this as a bug, please include the error ID below,
+        preferably by copying and pasting it rather than by taking a
+        screenshot.
+      </p>
+      <p>
         (Error <abbr>ID</abbr>:
         <tal:oops replace="structure view/oops_id_text" />)
       </p>