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

feat: add support for macos #156

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jointhejourney
Copy link

Hi there,

Our Flutter app, Journey Navigation, utilizes Amplitude for our iOS app and we wanted to use it for our MacOS app as well.

I took advantage of the sharedDarwinSource option, since the Amplitude-iOS package is compatible with MacOS. The changes are fairly straightforward, and everything seems to work ok.

Fixes #46

@jointhejourney
Copy link
Author

jointhejourney commented Oct 8, 2023

One thing I did notice, is that "Device Type" isn't available when running on MacOS. However, I don't think it has to do with this Flutter package, it might be an issue with Amplitude-iOS.

edit:

I tried overriding the event from the Flutter side of things by using the device_info_plus package + apple_product_name but couldn't get it to work. Seems like it can only be fixed by the Amplitude-iOS package.

@jointhejourney
Copy link
Author

@justin-fiedler any chance we could get this merged in?

@izaaz
Copy link

izaaz commented Feb 12, 2024

@jointhejourney Thanks for created this pull request. We are in the process of updating the flutter SDK to use our newer iOS and Android SDKs. So ideally, we'd want to use the macos support in our swift SDK. I'll keep this thread updated on an ETA on when this will be made available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flutter macOS Support
2 participants