Skip to content

Commit 2c9d206

Browse files
authored
Refactor properties in Mixpanel click tracking (#643)
* move expired flag as a top level property * Update trackLinkClick to not be dependant on appInsights
1 parent c0fff5e commit 2c9d206

File tree

5 files changed

+19
-9
lines changed

5 files changed

+19
-9
lines changed

AzureFunctions.Client/app/components/trial-expired.component.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,12 @@ export class TrialExpiredComponent implements OnInit {
2121

2222
ngOnInit() { }
2323

24-
trackClick(buttonName: string) {
24+
trackLinkClick(buttonName: string) {
2525
if (buttonName) {
2626
try {
27-
this._aiService.trackEvent(buttonName, { expired: "true" });
27+
this._aiService.trackLinkClick(buttonName, "true");
2828
} catch (error) {
29-
this._aiService.trackException(error, 'trackClick');
29+
this._aiService.trackException(error, 'trackLinkClick');
3030
}
3131
}
3232
}

AzureFunctions.Client/app/components/try-now.component.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -77,12 +77,12 @@ export class TryNowComponent implements OnInit {
7777
return n.length >= width ? n : new Array(width - n.length + 1).join(z) + n;
7878
}
7979

80-
trackClick(buttonName: string) {
80+
trackLinkClick(buttonName: string) {
8181
if (buttonName) {
8282
try {
83-
this._aiService.trackEvent(buttonName, {expired:this.trialExpired.toString()});
83+
this._aiService.trackLinkClick(buttonName, this.trialExpired.toString());
8484
} catch (error) {
85-
this._aiService.trackException(error, 'trackClick');
85+
this._aiService.trackException(error, 'trackLinkClick');
8686
}
8787
}
8888
}

AzureFunctions.Client/app/services/ai.service.ts

+10
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,16 @@ export class AiService implements IAppInsights {
134134
return appInsights.trackEvent(name, properties, measurements);
135135
}
136136

137+
/**
138+
* Log a user action or other occurrence.
139+
* @param name A string to identify this event in the portal.
140+
* @param expired string - determines if the link was clicked before or after the trial had expired .
141+
*/
142+
trackLinkClick(name: string, expired: string, properties?: { [name: string]: string; }, measurements?: { [name: string]: number; }) {
143+
if (typeof (mixpanel) !== 'undefined')
144+
mixpanel.track(name, { expired: expired, properties: this.addMixPanelProperties(null), measurements: measurements });
145+
}
146+
137147
/**
138148
* Log a dependency call
139149
* @param id unique id, this is used by the backend o correlate server requests. Use Util.newId() to generate a unique Id.

AzureFunctions.Client/templates/trial-expired.component.html

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ <h1>{{ 'trialExpired_enjoyedHostingFunctions' | translate }}</h1>
1414
<h3>{{ 'trialExpired_signupForAzure' | translate }}</h3>
1515
<div style="height:50px;"></div>
1616
<div>
17-
<a [attr.href]="freeTrialUri" (click)="trackClick('expiredFreeTrialClick')" target="_blank" class="trial-expired-signup-button">
17+
<a [attr.href]="freeTrialUri" (click)="trackLinkClick('expiredFreeTrialClick')" target="_blank" class="trial-expired-signup-button">
1818
{{ 'tryNow_createFreeAzureAccount' | translate }}
1919
</a>
2020
</div>

AzureFunctions.Client/templates/try-now.component.html

+2-2
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
</div>
66
<div>{{ 'tryNow_trialTimeRemaining' | translate }} <span id="timer-text">{{timerText}}</span></div>
77
<div class="free-trial-wrapper">
8-
<a [attr.href]="freeTrialUri" (click)="trackClick('freeTrialTopClick')" target="_blank" class="signup-button ">
8+
<a [attr.href]="freeTrialUri" (click)="trackLinkClick('freeTrialTopClick')" target="_blank" class="signup-button ">
99
{{ 'tryNow_createFreeAzureAccount' | translate }}
1010
</a>
1111
<i [tooltip]="freeAccountTooltip" class="fa fa-question-circle"></i>
@@ -16,7 +16,7 @@
1616
</tooltip-content>
1717
</div>
1818
<span class="discover-more-wrapper">
19-
<a [attr.href]="discoverMoreUri" (click)="trackClick('discoverMoreClick')" target="_blank" class="discover-more-button">
19+
<a [attr.href]="discoverMoreUri" (click)="trackLinkClick('discoverMoreClick')" target="_blank" class="discover-more-button">
2020
{{ 'tryNow_discoverMore' | translate }}
2121
</a>
2222
</span>

0 commit comments

Comments
 (0)