Closed
Bug 1525626
Opened 5 years ago
Closed 5 years ago
"event" ping docs incorrectly reference processStartTimestamp as processCreationTimestamp
Categories
(Toolkit :: Telemetry, enhancement, P3)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
mozilla67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: chutten, Assigned: ethompson, Mentored)
Details
(Whiteboard: [good first bug][lang=rst])
Attachments
(1 file)
https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/data/event-ping.html
There is no such thing as processCreationTimestamp
in "event" pings, only processStartTimestamp
Reporter | ||
Comment 2•5 years ago
|
||
To help Mozilla out with this bug, here's the steps:
- Comment here on the bug that you want to volunteer to help. I (or someone else) will assign it to you.
- Download and build the Firefox source code
- If you have any problems, please ask on IRC in the
#introduction
channel. They're there to help you get started. - You can also read the Developer Guide, which has answers to most development questions:
- If you have any problems, please ask on IRC in the
- Start working on this bug. You'll be editing the documentation at
toolkit/components/telemetry/docs/data/event-ping.rst
to readprocessStartTimestamp
instead ofprocessCreationTimestamp
.- If you have any problems with this bug, please comment on this bug and set the needinfo flag for me. Also, you can find me and my teammates on the #telemetry channel on IRC most hours of most days.
- Build your change with
mach doc
and then navigate to the documentation in the opened browser window to make sure it looks nice. For comparison, here's what it looks like now: https://firefox-source-docs.mozilla.org/toolkit/components/telemetry/telemetry/data/event-ping.html- Note:
mach doc
may not work especially well in Windows. If you'd like to contribute and are running Windows I can build the change on my machine and double-check it there.
- Note:
- Submit the patch for review. Mark me as a reviewer so I'll get an email to come look at your code.
- After a series of reviews and changes to your patch, I'll mark it for checkin or push it to autoland. Your contribution will soon be shipping to Firefox users worldwide!
- ...now you get to think about what kind of bug you'd like to work on next. Let me know what you're interested in and I can help you find your next contribution.
Mentor: chutten
Flags: needinfo?(chutten)
Whiteboard: [good first bug][lang=rst]
Assignee | ||
Comment 3•5 years ago
|
||
workin' on it!
Assignee | ||
Comment 4•5 years ago
|
||
changing processCreationTimestamp to processStartTimestamp
Updated•5 years ago
|
Assignee: nobody → ethompson
Pushed by chutten@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e141df747cab changing processCreationTimestamp to processStartTimestamp r=chutten
Comment 6•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•