Launchpad itself

Merge ~cjwatson/launchpad:please-copy-oops-ids into launchpad:master

  1. Git
  2. lp:~cjwatson/launchpad
  3. please-copy-oops-ids
  4. Merge into master
Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 3c5346e0ec89ff0b6d5101eadfaa2bed3a046588
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/launchpad:please-copy-oops-ids
Merge into: launchpad:master
Diff against target: 48 lines (+15/-0)
3 files modified
lib/lp/app/templates/launchpad-requestexpired.pt (+5/-0)
lib/lp/app/templates/oops.pt (+5/-0)
lib/lp/services/webapp/templates/oops-veryplain.pt (+5/-0)
Reviewer Review Type Date Requested Status
Kristian Glass (community) Approve
Tom Wardill (community) Approve
Review via email: mp+380353@code.launchpad.net

Commit message

Advise copying and pasting OOPS IDs

Description of the change

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.

To post a comment you must log in.
Revision history for this message
Tom Wardill (twom) :
review: Approve
Revision history for this message
Kristian Glass (doismellburning) wrote :

Even if it makes no difference, it's a cheap change - nice work

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/lib/lp/app/templates/launchpad-requestexpired.pt b/lib/lp/app/templates/launchpad-requestexpired.pt
2index 2afe81b..df768ca 100644
3--- a/lib/lp/app/templates/launchpad-requestexpired.pt
4+++ b/lib/lp/app/templates/launchpad-requestexpired.pt
5@@ -21,6 +21,11 @@
6 Trying again in a couple of minutes might work.
7 </p>
8 <p>
9+ If you report this as a bug, please include the error ID below,
10+ preferably by copying and pasting it rather than by taking a
11+ screenshot.
12+ </p>
13+ <p>
14 (Error <abbr>ID</abbr>:
15 <tal:oops replace="structure view/oops_id_text" />)
16 </p>
17diff --git a/lib/lp/app/templates/oops.pt b/lib/lp/app/templates/oops.pt
18index 82a88da..dcb9fa6 100644
19--- a/lib/lp/app/templates/oops.pt
20+++ b/lib/lp/app/templates/oops.pt
21@@ -32,6 +32,11 @@
22 Apologies for the inconvenience.
23 </p>
24 <p>
25+ If you report this as a bug, please include the error ID below,
26+ preferably by copying and pasting it rather than by taking a
27+ screenshot.
28+ </p>
29+ <p>
30 (Error <abbr>ID</abbr>:
31 <tal:oops replace="structure view/oops_id_text" />)
32 </p>
33diff --git a/lib/lp/services/webapp/templates/oops-veryplain.pt b/lib/lp/services/webapp/templates/oops-veryplain.pt
34index 797328e..ce05b72 100644
35--- a/lib/lp/services/webapp/templates/oops-veryplain.pt
36+++ b/lib/lp/services/webapp/templates/oops-veryplain.pt
37@@ -35,6 +35,11 @@
38 Apologies for the inconvenience.
39 </p>
40 <p>
41+ If you report this as a bug, please include the error ID below,
42+ preferably by copying and pasting it rather than by taking a
43+ screenshot.
44+ </p>
45+ <p>
46 (Error <abbr>ID</abbr>:
47 <tal:oops replace="structure view/oops_id_text" />)
48 </p>

Subscribers

People subscribed via source and target branches

to status/vote changes: