-
-
Notifications
You must be signed in to change notification settings - Fork 96
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
Feature: Any plans of adding Web Notification API integration? #59
Comments
I'd like to yes, but I've just started a new job and probably won't be using this for a while. So my feeling is that I won't be adding this, although I'd be open to helping someone with a PR. |
Awesome, congratulations on the new job. Thank you for all you do for the community. |
Thanks! That sounds really great to me. Let me know if there's anything I
can do to help.
|
Hi guys, I'll love see this feature here. I'm using ember-notify in my app and I need Web Notification support. I have same questions about it:
I can work on this pull request if is OK to @joshuairl |
Are we able to detect whether web notifications are enabled? If so then I think we should only display the web notification, not the DOM notification. I think it's important to add icon support for both DOM and web notifications. |
@aexmachina @dukex web notifications appear to need permission first. This other library (notifications.js) has some logic that calls the native API methods to determine whether the web notifications are enabled: |
@jfrux are you still interested in implementing native notifications? |
Just so that the browser is pinged if not in focus and stuff?
Just curious.
The text was updated successfully, but these errors were encountered: