Closed Bug 1364362 Opened 7 years ago Closed 7 years ago

Add doc link to error message when "record_in_processes" is missing

Categories

(Toolkit :: Telemetry, enhancement, P1)

Product:
Toolkit
Component:
Telemetry
Type:
enhancement
Points:
1

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla55
Tracking Flags:
Tracking Status
firefox55 --- fixed

People

(Reporter: gfritzsche, Assigned: gfritzsche)

References

Details

Attachments

(1 file, 2 obsolete files)

Add doc link for Histograms.json record_in_processes errors
4.13 KB, patch
gfritzsche
: review+
Details | Diff | Splinter Review 
Currently this prints:
> Error processing histograms:
> Histogram "A11Y_INSTANTIATED_FLAG" must have a "record_in_processes" field.

As this will be new for most developers, we should change this to something like:
> Error processing histograms:
> Histogram "A11Y_INSTANTIATED_FLAG" must have a "record_in_processes" field.
> See: http://gecko.readthedocs.io/en/latest/toolkit/components/telemetry/telemetry/collection/histograms.html#record-in-processes
Attachment #8867090 - Attachment is obsolete: true
Attachment #8867090 - Flags: review?(chutten)
Attachment #8867124 - Flags: review?(chutten) → review+
Attachment #8867124 - Attachment is obsolete: true
Attachment #8867217 - Flags: review+
Keywords: checkin-needed
Needs rebasing.
Keywords: checkin-needed
nvm, just needed to land on top of bug 1364393
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c60f2a5a69af
Add doc link for Histograms.json record_in_processes errors. r=chutten
https://hg.mozilla.org/mozilla-central/rev/c60f2a5a69af
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: