-
Notifications
You must be signed in to change notification settings - Fork 34
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
Android Oreo NotificationChannel support #97
Comments
I have verified this, and it's indeed an issue. I've patched it for now, but the best thing would be to expose the full functionality of the channels. |
How did you fix? |
I'll submit a PR so you can check it out, but It's not the optimal solution for the use of channels. |
I added a (very :) ) little support for Oreo: #102 |
That's pretty much how I did it @slash-84 |
Hello to all. |
Hello |
Any plans for a new release to fully support Notifications channels any time soon? Starting August new apps submitted to PlayStore have to target SDK 26, so the module won't be usable any more. Thank you for your work so far |
In Ti-Slack someone mentioned, that there could be problems with push under Oreo: missing NotificationChannel. It is related to this SO thread:
https://stackoverflow.com/questions/45711925/failed-to-post-notification-on-channel-null-target-api-is-26
Since I don't have an Oreo phone yet I can't prove this but it something that should be on the roadmap if the problem exists in Ti.Goosh too
The text was updated successfully, but these errors were encountered: