Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #26902 from edx/ddumesnil/segment-logout-aa-513

Browse Source 
AA-513: Reset segment state if anon user and there is a segment user id
This commit is contained in:
Dillon Dumesnil
2021-03-10 11:08:16 -08:00
committed by GitHub
parent dabfa5ff08 16d81f6e6d
commit e641ba368c
2 changed files with 32 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
cms/templates/widgets/segment-io-footer.html
View File

@@ -30,6 +30,22 @@
else {
onLoadCallback();
}
% else:
// If we do not have an authenticated user, but Segment has a user id,
// reset the Segment user state so we start tracking fresh.
// This has to be wrapped in the document.readyState logic because the analytics.user()
// function isn't available until the analytics.js package has finished initializing.
if (document.readyState === 'complete') {
if (analytics.user().id()) {
analytics.reset();
}
} else {
document.addEventListener('readystatechange', event => {
if (event.target.readyState === 'complete' && analytics.user().id()) {
analytics.reset();
}
});
}
% endif
</script>
<!-- end segment footer -->

16
lms/templates/widgets/segment-io-footer.html
View File

@@ -22,6 +22,22 @@
}
);
});
% else:
// If we do not have an authenticated user, but Segment has a user id,
// reset the Segment user state so we start tracking fresh.
// This has to be wrapped in the document.readyState logic because the analytics.user()
// function isn't available until the analytics.js package has finished initializing.
if (document.readyState === 'complete') {
if (analytics.user().id()) {
analytics.reset();
}
} else {
document.addEventListener('readystatechange', event => {
if (event.target.readyState === 'complete' && analytics.user().id()) {
analytics.reset();
}
});
}
% endif
</script>
<!-- end segment footer -->