LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Revert "check matomo and disable if necessary" #4655

Merged
jessopb merged 1 commit from revert-4654-detectMatomoOutage into master 2020-08-12 17:04:55 +02:00
Conversation 0 Commits 1 Files changed 1 +2 -19
1 changed files with 2 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
ui/analytics.js
View file

@ -56,24 +56,7 @@ type LogPublishParams = {
channel_claim_id?: string,
};
const checkmatomo = (url, timeout = 2000) => {
return Promise.race([
fetch(url),
new Promise((resolve, reject) => setTimeout(() => reject(new Error('timeout')), timeout)),
]);
};
let internalAnalyticsEnabled: boolean = IS_WEB || false;
let matomoOnline = true;
if (internalAnalyticsEnabled) {
checkmatomo(MATOMO_URL)
.then()
.catch(() => {
matomoOnline = false;
console.log('matomo offline');
});
}
// let thirdPartyAnalyticsEnabled: boolean = IS_WEB || false;
// @if TARGET='app'
if (window.localStorage.getItem(SHARE_INTERNAL) === 'true') internalAnalyticsEnabled = true;
@ -106,7 +89,7 @@ const analytics: Analytics = {
});
},
pageView: path => {
if (internalAnalyticsEnabled && matomoOnline) {
if (internalAnalyticsEnabled) {
MatomoInstance.trackPageView({
href: `${path}`,
});
@ -262,7 +245,7 @@ const analytics: Analytics = {
};
function sendMatomoEvent(category, action, name, value) {
if (internalAnalyticsEnabled && matomoOnline) {
if (internalAnalyticsEnabled) {
const event = { category, action, name, value };
MatomoInstance.trackEvent(event);
}