Closed
Bug 1734731
Opened 4 years ago
Closed 4 years ago
Glean.shutdown in qt errors.
Categories
(Data Platform and Tools Graveyard :: Glean.js, defect, P1)
Data Platform and Tools Graveyard
Glean.js
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sarah, Assigned: brizental)
Details
Attachments
(1 file)
Warning: qrc:/glean/org/mozilla/Glean/glean.js:102: TypeError: Property 'shutdown' of object [object Object] is not a function (glean.js:102)
The function in glean.js is
/**
* Finishes executing any ongoing tasks and shuts down Glean.
*
* This will attempt to send pending pings before resolving.
*
* @returns A promise which resolves once shutdown is complete.
*/
function shutdown() {
return Glean.shutdown();
}
I have a suspicion it should be Glean.Glean.default.shutdown() in the way that Glean.initialize looks like this
/**
* Initialize Glean. This method should only be called once, subsequent calls will be no-op.
*
* @param applicationId The application ID (will be sanitized during initialization).
* @param uploadEnabled Determines whether telemetry is enabled.
* If disabled, all persisted metrics, events and queued pings (except
* first_run_date) are cleared.
* @param config Glean configuration options.
*/
function initialize(applicationId, uploadEnabled, config) {
Glean.Glean.default.initialize(applicationId, uploadEnabled, config);
}
| Assignee | ||
Comment 1•4 years ago
|
||
I have a suspicion it should be Glean.Glean.default.shutdown() in the way that Glean.initialize looks like this
Yes you are 100% right here. Woof that was an ooversight, I will have a fix later today.
Assignee: nobody → brizental
Priority: P4 → P1
Whiteboard: [telemetry:glean-js:m?]
| Assignee | ||
Comment 2•4 years ago
|
||
| Assignee | ||
Comment 3•4 years ago
|
||
Work on Bug 1708194 uncovered this issue as well so I applied the fixes for it there.
| Assignee | ||
Updated•4 years ago
|
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Updated•3 years ago
|
Product: Data Platform and Tools → Data Platform and Tools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•