Closed
Bug 801938
Opened 12 years ago
Closed 12 years ago
Crash reports information page for "What's in a crash report?" and "Learn More" links
Categories
(Firefox OS Graveyard :: Gaia, defect, P1)
Firefox OS Graveyard
Gaia
Tracking
(blocking-basecamp:+)
People
(Reporter: lco, Assigned: Margaret)
References
()
Details
(Keywords: feature, l12y)
Attachments
(1 file)
See pg. 9 of the spec: http://people.mozilla.com/~lco/Crash_Reporting_B2G/
The user can access more information about what a crash report contains and what Mozilla does with a crash report from two places:
1. The Crash Reporting Prompt (link label is "What's in a crash report?"
2. The "Improve Firefox OS" Settings Page (link label is "Learn More")
Reporter | ||
Comment 1•12 years ago
|
||
Alina/Tom, are you guys the people to ping about writing the contents of this dialog? If you aren't, who is?
Updated•12 years ago
|
Blocks: b2g-crash-reporting
Tom - Thanks for working on this.
Larissa - When we chatted yesterday, you mentioned the work was just starting on crash reporting, so if a draft of the copy is sent to you early next week, would that work for you?
Reporter | ||
Comment 3•12 years ago
|
||
(In reply to Alina Hua from comment #2)
> Larissa - When we chatted yesterday, you mentioned the work was just
> starting on crash reporting, so if a draft of the copy is sent to you early
> next week, would that work for you?
I think this is ok. Thanks for working on this, Tom!
Assignee | ||
Comment 4•12 years ago
|
||
Clarifying the summary of this bug.
The "What's in a crash report?" link will be in the crash report dialog.
The "Learn More" link will be in the settings UI.
Summary: As a user, I want to understand what a crash report contains and how Mozilla uses this information, so that I know why Mozilla is asking me whether I want to report a crash → Crash reports information page for "What's in a crash report?" and "Learn More" links
Assignee | ||
Comment 5•12 years ago
|
||
Checking in to see how progress is coming on this bug. The crash reporter dialog UI landed today, so we'd be ready to implement a page for the "Learn More..." link.
Nominating this for blocking because without it the links in these mockups will do nothing.
blocking-basecamp: --- → ?
Comment 6•12 years ago
|
||
New feature work for v1. Marking as blocking.
Tom/Alina, do you guys have an update on the text here for the two links? Thanks.
blocking-basecamp: ? → +
Priority: -- → P1
Reporter | ||
Comment 7•12 years ago
|
||
Spoke to Tom yesterday, he is on it :)
Assignee: nobody → margaret.leibovic
Assignee | ||
Comment 8•12 years ago
|
||
Larissa, right now the "Learn More" link for the "Performance Data" pref opens a webpage in the browser. Should we also make this a webpage? Or should we file a bug to change the behavior of the other link? It feels odd that they would behave differently.
Reporter | ||
Comment 9•12 years ago
|
||
I(In reply to Margaret Leibovic [:margaret] from comment #8)
> Larissa, right now the "Learn More" link for the "Performance Data" pref
> opens a webpage in the browser. Should we also make this a webpage? Or
> should we file a bug to change the behavior of the other link? It feels odd
> that they would behave differently.
I can file a bug for "Learn More" in the "Performance Data" page. That should be an internal site as well since the text is likely not to change very much.
Assignee | ||
Comment 10•12 years ago
|
||
Making these into local pages was marked as blocking- in bug 807187, so maybe we should just make this into a webpage for now. Is there an existing webpage that we would be able to direct users to, at least for the time being?
Comment 11•12 years ago
|
||
We're marking this bug with the C1 milestone since it follows the criteria of "unfinished feature work" (see https://etherpad.mozilla.org/b2g-convergence-schedule).
If this work is not finished by Nov19, this bug will need an exception and will be called out at the upcoming Exec Review.
Target Milestone: --- → B2G C1 (to 19nov)
Assignee | ||
Comment 12•12 years ago
|
||
Dietrich and I talked about this, and using a webpage, similarly to how we do in-product links in Firefox would be the most straightforward way to resolve the blocking issue here.
To do this, I'll just need a URL to point these two links to. For desktop/Android, we point users to SUMO kb articles for things like these (e.g. https://support.mozilla.org/en-US/kb/send-performance-data-improve-firefox). Can we do that for b2g?
I still haven't heard back about the actual text for this page, but that could just be included in a kb article. I'd just need to know what URL to use, and the preferred way of doing that in Gaia. Do we need to make it a localized string, or can SUMO magic just redirect to the correct locale?
Assignee | ||
Comment 13•12 years ago
|
||
Poking this bug, since it's a C1 blocker.
Still need strings here, or a link to a SUMO kb article (preferred method).
mluna, is there a SUMO base URL for Firefox OS? If so, I can just make up a URL to a kb article, similarly to how we did for telemetry "Learn More" links for desktop/Android.
Reporter | ||
Comment 14•12 years ago
|
||
Alina says Tom will provide the string once he gets back from MozFest. Tom, ETA?
Flags: needinfo?(tom)
Assignee | ||
Comment 15•12 years ago
|
||
(In reply to Margaret Leibovic [:margaret] from comment #13)
> Still need strings here, or a link to a SUMO kb article (preferred method).
>
> mluna, is there a SUMO base URL for Firefox OS? If so, I can just make up a
> URL to a kb article, similarly to how we did for telemetry "Learn More"
> links for desktop/Android.
Hrm, I just realized that this SUMO link solution won't work for the dialog, since clicking the link requires opening Firefox, but I don't think we can do that from a modal dialog.
(In reply to Larissa Co from comment #14)
> Alina says Tom will provide the string once he gets back from MozFest. Tom,
> ETA?
I'll work on the logic for making a "Crash Reports" page, but I need to know what to put in it by the end of the week.
Cc'ing stas, as this will involve strings if we don't go with the SUMO link approach.
Assignee | ||
Comment 16•12 years ago
|
||
I started working on a patch for the settings part of this here:
https://github.com/leibovic/gaia/commit/5bf16c8132bce0ca15b5ef49075d21169a4a7461
Kaze, I'm seeing a bunch more of those "Warning: Empty string passed to getElementById()" messages, like in bug 811447. Is there something I'm doing wrong here? Obviously I don't have the real string here, but if you can offer me feedback that would speed up review later! :)
Assignee | ||
Comment 17•12 years ago
|
||
Here's my current WIP for the crash reports page from the dialog:
https://github.com/leibovic/gaia/commit/04d2617c518797b2128c00cbbcb6c34fad757641
These two pages in settings/system look slightly different, but I feel like that's okay as long as they're consistent within the context where they're shown. Also, they definitely look okay enough for landing to fix this blocker :)
Once I get the final strings I'll finish up these patches and request review!
Assignee | ||
Comment 18•12 years ago
|
||
I opened a PR, but I'll wait to get the final strings before flagging for review.
Assignee | ||
Comment 19•12 years ago
|
||
Comment on attachment 682504 [details]
Link to https://github.com/mozilla-b2g/gaia/pull/6465
I filed bug 813128 to update the string with the final text, so we can just land this today.
Attachment #682504 -
Flags: review?(kaze)
Attachment #682504 -
Flags: review?(alive)
Comment 20•12 years ago
|
||
Comment on attachment 682504 [details]
Link to https://github.com/mozilla-b2g/gaia/pull/6465
Looks good, thanks!
Attachment #682504 -
Flags: review?(kaze) → review+
Assignee | ||
Updated•12 years ago
|
Attachment #682504 -
Flags: review?(alive)
Assignee | ||
Comment 21•12 years ago
|
||
https://github.com/mozilla-b2g/gaia/commit/75f0bf4450be8038ff3e98a3cd1db922f023264d
Follow-up strings to come in bug 813128.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Flags: needinfo?(tom)
You need to log in
before you can comment on or make changes to this bug.
Description
•