-
Notifications
You must be signed in to change notification settings - Fork 711
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
[Bug 1083434] Send notifications to SimplePush. #2346
[Bug 1083434] Send notifications to SimplePush. #2346
Conversation
Docs about Simple Push are here, though they are very light on the server side: https://developer.mozilla.org/en-US/docs/Web/API/Simple_Push_API |
Test it on stage/prod? This all looks good to me. If it is broken, only celery tasks should be affected so no biggie. r+ |
We probably can't even test this until BuddyUp writes some stuff. Chicken and egg, I guess. We can push this to stage and stare at it for a bit to make sure nothing blows up. |
[Bug 1083434] Send notifications to SimplePush.
I don't really have a good way to test this. I can't run a simple push server, and I don't think anything but a Firefox OS phone can run the client side of this. I could probably code up a simple test instance, but I don't really want to. I wrote tests that assert at least we call the URL that is registered.
r?