Skip to content

Commit

Permalink
fix: segment plugin use local initialized variable
Browse files Browse the repository at this point in the history
  • Loading branch information
bgiori committed Feb 14, 2025
1 parent 5d6f5d1 commit 0c629f6
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions packages/plugin-segment/src/plugin.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,16 +15,22 @@ export const segmentIntegrationPlugin: SegmentIntegrationPlugin = (
return options.instance || snippetInstance(options.instanceKey);
};
getInstance();
let initialized = false;
const plugin: IntegrationPlugin = {
name: '@amplitude/experiment-plugin-segment',
type: 'integration',
setup(): Promise<void> {
const instance = getInstance();
return new Promise<void>((resolve) => instance.ready(() => resolve()));
return new Promise<void>((resolve) =>
instance.ready(() => {
initialized = true;
resolve();
}),
);
},
getUser(): ExperimentUser {
const instance = getInstance();
if (instance.initialized) {
if (initialized) {
return {
user_id: instance.user().id(),
device_id: instance.user().anonymousId(),
Expand All @@ -42,7 +48,7 @@ export const segmentIntegrationPlugin: SegmentIntegrationPlugin = (
},
track(event: ExperimentEvent): boolean {
const instance = getInstance();
if (!instance.initialized) return false;
if (!initialized) return false;
instance.track(event.eventType, event.eventProperties);
return true;
},
Expand Down

0 comments on commit 0c629f6

Please sign in to comment.