Update the Amplitude integration to Amplitude's Browser SDK 2

optimizely / library

Optimizely Library

Other

74 stars 70 forks source link

Update the Amplitude integration to Amplitude's Browser SDK 2 #84

Closed MarkBiesheuvel closed 2 months ago

MarkBiesheuvel commented 2 months ago

A customer reported issues with the Amplitude integration. They are using the Amplitude's Browser SDK 2, which does not seem to have the getInstance function. Therefore, I have rewritten the integration use the syntax using the v2 documentation. https://amplitude.com/docs/sdks/analytics/browser/browser-sdk-2#user-properties I have verified with the customer that the modified integration works.

MarkBiesheuvel commented 2 months ago

@simone-coelho Could you review this pull request? Thank you

simone-coelho commented 2 months ago

@MarkBiesheuvel thanks for doing this!