Skip to content

add support for mixpanel for omi macOS; automatic events included#3484

Merged
beastoin merged 1 commit intomainfrom
i8i57_mixpanel_macos
Nov 22, 2025
Merged

add support for mixpanel for omi macOS; automatic events included#3484
beastoin merged 1 commit intomainfrom
i8i57_mixpanel_macos

Conversation

@beastoin
Copy link
Copy Markdown
Collaborator

@beastoin beastoin commented Nov 22, 2025

what's included?

  • tracking the automatic events ($ae_first_app_open, $ae_sesison, etc...) for omi macos

deps:

demo: #3484 (comment)

deploy:

  • deploy macos app

notices: the macos lib is not officially supported by the mixpanel sdk; use it at your own risk.

#3455

@beastoin
Copy link
Copy Markdown
Collaborator Author

Screenshot 2025-11-22 at 18 57 10 Screenshot 2025-11-22 at 18 54 31 Screenshot 2025-11-22 at 17 26 05

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds Mixpanel support for macOS by leveraging a forked version of mixpanel-flutter that includes native Swift bindings. The platform detection logic has been updated to use mixpanel-flutter on macOS, iOS, and Android, while falling back to mixpanel_analytics for other platforms like Windows. The changes look good and align with the goal of the PR. I've left one comment regarding the use of an SSH URL for a pod dependency in the Podfile, which should be changed to HTTPS for better build reliability.

Comment thread app/macos/Podfile
@beastoin beastoin merged commit 306ff00 into main Nov 22, 2025
1 check passed
@beastoin beastoin deleted the i8i57_mixpanel_macos branch November 22, 2025 12:15
Glucksberg pushed a commit to Glucksberg/omi-local that referenced this pull request Apr 28, 2026
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.

1 participant