-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Platform reports iOS
while running on macOS
#31
Comments
Thanks for this report @AvdLee. I'll add a ticket for updating this currently hardcoded field |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
See this line of code: https://github.com/amplitude/experiment-ios-client/blob/main/Sources/Experiment/DefaultUserProvider.swift#L54
When using the SDK on macOS, I would expect this value to be
macOS
.The Amplitude SDK seems to do this differently, since our analytics still show users for
macOS
. However, variants are fetched usingiOS
instead:Note the value for
os
beingios 13.3.1.
: the macOS version is correct, but it's prefixed withiOS
instead ofmacOS
. Due to this, our experiment didn't work as we were targetingmacOS
users.The text was updated successfully, but these errors were encountered: