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

Headup not displayed for local notification of oreo and above version. #1879

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

AshwiniKhandagale
Copy link

I am facing the issue of headUp was not display for Android 8 or above version, Then I have review the plugin I saw one mistake. Priority is set to IMPORTANCE_DEFAULT. I have to make a small change priority set to IMPORTANCE_HIGH.

@CodeWithOz
Copy link

@katzer thanks a lot for all your hard work. Is there any chance of this getting merged? Or if there's a problem with the PR can you please give some feedback on what needs to be changed? Thanks!

@CodeWithOz
Copy link

@AshwiniKhandagale thanks for the PR, it's very useful. I've seen that you use a default channel called "VirGo". I have a number of channels that my app uses so that single one won't work for me. What do you suggest? I've forked your branch and I'm considering just making the channel names dynamic (based on values I supply), but do you have any suggestions? Or have you solved that problem in a different way? Thanks!

@rahmadid
Copy link

@CodeWithOz Hi, perhaps you can try this fork, it is more recent and still maintained (to my knowledge):
https://github.com/timkellypa/cordova-plugin-local-notifications

This fork also includes updates for Android 10.

Hope it helps. Thanks.

@CodeWithOz
Copy link

@rahmadid thanks yeah it's doing what I want it to do, thanks for pointing me in that direction!

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.

3 participants