Skip to content
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

check for ios app on mac in automatic props #521

Merged
merged 7 commits into from
Apr 11, 2022

Conversation

jaredmixpanel
Copy link
Contributor

@jaredmixpanel jaredmixpanel commented Mar 3, 2022

This PR adds logic to check if the app is a "Designed for iPad" app running on macOS and if so, sets the $os to macOS. Unfortunately, there is no API that reports the correct macOS version for "Designed for iPad" apps running on macOS, so we omit it rather than mis-report

@jaredmixpanel jaredmixpanel requested a review from zihejia March 3, 2022 20:00
@jaredmixpanel jaredmixpanel changed the title [WIP] check for ios app on mac in automatic props check for ios app on mac in automatic props Apr 11, 2022
Copy link
Contributor

@zihejia zihejia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jaredmixpanel jaredmixpanel merged commit e3feb51 into master Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants