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

Browser Notifications #229

Closed
mpfau opened this issue Jan 3, 2018 · 3 comments
Closed

Browser Notifications #229

mpfau opened this issue Jan 3, 2018 · 3 comments
Labels
new feature user stories in varying refinement states state:tested We tested it and are about to release it

Comments

@mpfau
Copy link
Contributor

mpfau commented Jan 3, 2018

No description provided.

@charlag
Copy link
Contributor

charlag commented Aug 1, 2018

ServiceWorker doesn't live long enough to be used with notifications. In fact, SSE is even not usually exposed in the workers: w3c/ServiceWorker#947

What I believe we need is web push: https://developer.mozilla.org/en-US/docs/Web/API/Push_API
"The service worker will be started as necessary to handle incoming push messages" which sounds like our case.

It requires additional work on the server but I think the implementation will be trivial.

@armhub armhub added the new feature user stories in varying refinement states label Sep 20, 2018
@mpfau mpfau modified the milestones: Roadmap, Out of beta Oct 16, 2018
@bedhub bedhub modified the milestones: Out of beta, Next release Oct 24, 2018
@charlag charlag changed the title Browser Notifications (ServiceWorker) Browser Notifications Oct 25, 2018
@bedhub bedhub closed this as completed Oct 25, 2018
@asteintutao
Copy link

Tested on Chrome, Opera and Firefox on Windows, seems to work fine. The pop up window is showing it self in the bottom right corner overlaying the newMailActionButton

@charlag charlag added the state:tested We tested it and are about to release it label Oct 25, 2018
@charlag
Copy link
Contributor

charlag commented Oct 26, 2018

It works in mobile Firefox too but not in mobile Chrome (they probably want service worker implementation).
Icon doesn't work with Chrome on Linux and with Safari but it seems like it's not our fault

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature user stories in varying refinement states state:tested We tested it and are about to release it
Projects
None yet
Development

No branches or pull requests

5 participants