-
-
Notifications
You must be signed in to change notification settings - Fork 122
Re-implement iOS native plugin (+ minor features) #342
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
Conversation
Hello guys, Any news on this PR ? |
Hello, I tested the PR in prod and I have these issues: Fatal Exception: android.app.RemoteServiceException$ForegroundServiceDidNotStartInTimeException: Context.startForegroundService() did not then call Service.startForeground(): ServiceRecord{fd785c9 u0 com.wayup.way_up/com.gdelataillade.alarm.alarm.AlarmService}
|
On the iOS side, the alarm sounds fine and I no longer have any crashes. The only downside is the notifications issue: I still can't use a payload on my notifications when the package is active. @orkun1675 Congratulations on the PR work! |
Merging this PR. @wayupdev can you please create bugs for the two issues you mentioned? |
This XXL PR achieves the following:
AlarmSet
class used byscheduled
andringing
streamsiOSTimers
that are a fallback mechanism incase native->Flutter communication breaks downMissing pieces include: