Skip to content

Commit 2ede4e8

Browse files
committed
updated UTM params handling
1 parent 9612942 commit 2ede4e8

File tree

1 file changed

+14
-15
lines changed

1 file changed

+14
-15
lines changed

packages/sdk/src/index.ts

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -75,15 +75,15 @@ export class MermaidChart {
7575
public async getAuthorizationData({
7676
state,
7777
scope = ['email'],
78-
utm_source,
79-
utm_medium,
80-
utm_campaign
78+
trackingParams,
8179
}: {
8280
state?: string;
8381
scope?: string[];
84-
utm_source?: string;
85-
utm_medium?: string;
86-
utm_campaign?: string;
82+
trackingParams?: {
83+
utm_source: string;
84+
utm_medium: string;
85+
utm_campaign: string;
86+
}
8787
} = {}): Promise<AuthorizationData> {
8888
if (!this.redirectURI) {
8989
throw new Error('redirectURI is not set');
@@ -96,20 +96,19 @@ export class MermaidChart {
9696
this.pendingStates[stateID] = {
9797
codeVerifier,
9898
};
99-
100-
const extraParams: Record<string, string> = {};
101-
if (utm_source && utm_medium && utm_campaign) {
102-
extraParams.utm_source = utm_source;
103-
extraParams.utm_medium = utm_medium;
104-
extraParams.utm_campaign = utm_campaign;
105-
}
10699

107100
const url = await this.oauth.authorizationCode.getAuthorizeUri({
108101
redirectUri: this.redirectURI,
109102
state: stateID,
110103
codeVerifier,
111104
scope,
112-
...(Object.keys(extraParams).length > 0 && { extraParams }),
105+
...(trackingParams && {
106+
extraParams: {
107+
utm_source: trackingParams.utm_source,
108+
utm_medium: trackingParams.utm_medium,
109+
utm_campaign: trackingParams.utm_campaign,
110+
}
111+
}),
113112
});
114113

115114
// Deletes the state after 60 seconds
@@ -267,4 +266,4 @@ export class MermaidChart {
267266
const raw = await this.axios.get<string>(URLS.raw(document, theme).svg);
268267
return raw.data;
269268
}
270-
}
269+
}

0 commit comments

Comments
 (0)