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

Delayed notifications firing immediately #35

Open
mibrahim025 opened this issue Jan 18, 2022 · 3 comments
Open

Delayed notifications firing immediately #35

mibrahim025 opened this issue Jan 18, 2022 · 3 comments

Comments

@mibrahim025
Copy link

Can you please fix the issue of delayed notifications firing immediately.. are you aware of that?

@d7laungani
Copy link
Owner

do you mind giving me a code snippet that you set up so I can look into this? Thanks

@higalex
Copy link

higalex commented Feb 12, 2022

Hi @d7laungani , so, assuming you create a notification that repeats every day, I think the date components would look like this:

calendar.components([.hour, .minute], from: date)

Even if this date is a date from next month, the notification will start to get fired from tomorrow (or even today if the current time is before the specified hour) because the day info gets cut off by the date components.

In order for it to work properly, you'd have to do some complicated tricks like, create a non-repeating notification for next month first and then create another repeating one for after the firing date.

I know it's stupid and Apple should really do something about it, but it has been that way for many years now. Unless that is taken care of, I honestly don't feel much benefit to use this library.

@mibrahim025
Copy link
Author

mibrahim025 commented Mar 16, 2022 via email

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

No branches or pull requests

3 participants