Closed
Bug 1783722
Opened 3 years ago
Closed 3 years ago
Structured `default_browser` pings missing `notification_not_shown_reason` column
Categories
(Data Platform and Tools :: General, defect, P4)
Data Platform and Tools
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: efilho, Unassigned)
References
Details
(Whiteboard: [dataquality])
Attachments
(1 obsolete file)
No description provided.
|
||
Comment 1•3 years ago
|
||
Kirk, can you comment on this? Does this need to be added to the ping schema?
Flags: needinfo?(bytesized)
|
||
Comment 2•3 years ago
|
||
I guess? notification_not_shown_reason was added to the default browser agent's pings in Bug 1776069. I looked through the tree for a schema to add it to when I wrote the patch, but I didn't see one.
Flags: needinfo?(bytesized)
|
|
||
Comment 3•3 years ago
|
||
Got it. For this to be made available in the table the new field has to be added to the schema here.
Flags: needinfo?(bytesized)
Priority: -- → P4
|
||
Comment 4•3 years ago
|
||
|
|
||
Updated•3 years ago
|
Flags: needinfo?(bytesized)
|
|
||
Comment 5•3 years ago
|
||
Gah, moved to https://github.com/mozilla-services/mozilla-pipeline-schemas/pull/744 because I suck at github
|
|
||
Updated•3 years ago
|
Attachment #9290484 -
Attachment is obsolete: true
|
|
||
Comment 6•3 years ago
|
||
PR 744 merged.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
|
||
Updated•2 years ago
|
Whiteboard: [data-quality] → [dataquality]
You need to log in
before you can comment on or make changes to this bug.
Description
•