Closed
Bug 1519402
Opened 6 years ago
Closed 6 years ago
Properly implement the "seq" field in the "ping_info" section
Categories
(Toolkit :: Telemetry, enhancement, P1)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox66 | --- | affected |
People
(Reporter: Dexter, Assigned: mdroettboom)
References
Details
(Whiteboard: [telemetry:mobilesdk:m4])
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
Details |
While implementing other metrics, we skipped on implementing the sequence numbers for pings: https://github.com/mozilla-mobile/android-components/blob/ca697d86d1c70d1e3e159480b45f87b9b75515c8/components/service/glean/src/main/java/mozilla/components/service/glean/ping/PingMaker.kt#L64
The spec says this field is "A running counter of the number of times this ping type has been sent".
Reporter | ||
Updated•6 years ago
|
Assignee: nobody → alessio.placitelli
Blocks: 1491345
Points: --- → 2
Priority: -- → P1
Whiteboard: [telemetry:mobilesdk:m4]
Assignee | ||
Updated•6 years ago
|
Assignee: alessio.placitelli → mdroettboom
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•