Always expect generationTimeMs
This commit is contained in:
+4
-3
@@ -148,6 +148,9 @@ class Analytics {
|
|||||||
}
|
}
|
||||||
|
|
||||||
trackPageChange(generationTimeMs) {
|
trackPageChange(generationTimeMs) {
|
||||||
|
if (typeof generationTimeMs !== 'number') {
|
||||||
|
throw new Error('Analytics.trackPageChange: expected generationTimeMs to be a number');
|
||||||
|
}
|
||||||
if (this.disabled) return;
|
if (this.disabled) return;
|
||||||
if (this.firstPage) {
|
if (this.firstPage) {
|
||||||
// De-duplicate first page
|
// De-duplicate first page
|
||||||
@@ -156,9 +159,7 @@ class Analytics {
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
this._paq.push(['setCustomUrl', getRedactedUrl()]);
|
this._paq.push(['setCustomUrl', getRedactedUrl()]);
|
||||||
if (typeof generationTimeMs === 'number') {
|
this._paq.push(['setGenerationTimeMs', generationTimeMs]);
|
||||||
this._paq.push(['setGenerationTimeMs', generationTimeMs]);
|
|
||||||
}
|
|
||||||
this._paq.push(['trackPageView']);
|
this._paq.push(['trackPageView']);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user