-
Notifications
You must be signed in to change notification settings - Fork 72
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
Made some modifications on the IOS implementation to unify behavior with Android’s one. #51
Conversation
…ation into benjamindean-master
Thank you for the contribution! Could you, please, format the code so it's easier to see the diff? |
Yes, sorry about that was just happy I got it to work and forgot to clean the code! |
Thanks! I will try to test it as soon as I can. |
Any update? |
Hey @benjamindean ! Just saw that you've been worked on this project recently 😁 |
Hey! Yeah, I guess I cn safely merge it now. Thanks for notifying! |
Will release a new version as soon as the web implementation is fixed. |
Only thing not working now on iOS is the vibration without Amplitude (only pattern). I unfortunately don’t have access to iOS hardware anymore so cannot implement that.
Amplitudes is confirmed to work and the test app works similarly on iOS and Android.