Closed Bug 801938 Opened 8 years ago Closed 8 years ago
Crash reports information page for "What's in a crash report?" and "Learn More" links
246 bytes, text/html
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")
Alina/Tom, are you guys the people to ping about writing the contents of this dialog? If you aren't, who is?
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?
(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!
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
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: --- → ?
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
Spoke to Tom yesterday, he is on it :)
Assignee: nobody → margaret.leibovic
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(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.
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?
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)
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?
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.
Alina says Tom will provide the string once he gets back from MozFest. Tom, ETA?
(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.
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! :)
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!
I opened a PR, but I'll wait to get the final strings before flagging for review.
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.
Comment on attachment 682504 [details] Link to https://github.com/mozilla-b2g/gaia/pull/6465 Looks good, thanks!
Attachment #682504 - Flags: review?(kaze) → review+
https://github.com/mozilla-b2g/gaia/commit/75f0bf4450be8038ff3e98a3cd1db922f023264d Follow-up strings to come in bug 813128.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
verified on unagi build id: 20130103070201
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.