Skip to content
This repository has been archived by the owner on Aug 24, 2021. It is now read-only.

[PWA] Push notification plugins support #3

Open
3 tasks
pradeep910 opened this issue Dec 19, 2019 · 4 comments
Open
3 tasks

[PWA] Push notification plugins support #3

pradeep910 opened this issue Dec 19, 2019 · 4 comments
Assignees
Labels
enhancement New feature or request planning Planning of a feature/task

Comments

@pradeep910
Copy link
Collaborator

pradeep910 commented Dec 19, 2019

Overview

There are multiple third-party plugins available for Push notifications. Such as Izooto, Onesignal
See the complete list here -
https://www.wpexplorer.com/wordpress-push-notification-plugins/
We need to add a support for these plugins/services.

Goal

Have an option on plugin settings page where user can select which push notification service to use. Either OneSignal or iZooto or some other.

Tasks

  • Create an option (radio button) to choose push notification service
  • Any additional fields for secret-keys or api keys that push notifications plugin would need.

**updated

  • Web push notification support for new posts without node/thirdparty services.
@pradeep910
Copy link
Collaborator Author

@deepaklalwani97

Pls check this plugin and see how it works - https://wordpress.org/plugins/progressive-wp/

@pradeep910 pradeep910 added the planning Planning of a feature/task label Dec 19, 2019
@deepaklalwani97
Copy link
Member

@deepaklalwani97

Pls check this plugin and see how it works - https://wordpress.org/plugins/progressive-wp/

I have checked this plugin it is based on some older versions of the PWA plugin. It provides features like offline page access, push notifications with firebase and integration with one signal plugin but had lot of issues like it uses older hooks to register script which might make extensions using new hooks incompatible and the integration with one signal does not work properly and had some errors.

@deepaklalwani97
Copy link
Member

deepaklalwani97 commented Dec 23, 2019

I tried to integrate izooto push notification service with the PWA plugin. The izooto provides very little control over which service worker to use in both the plugin and the manual integration method as it uses CDN to add its code for the service workers and SDK which results in conflict with the PWA plugin service worker. The solution for this could be to disable the parse request for URL which echoes the service worker code and the notifications work properly but doing this will throw some errors in the console as the izooto SDK expects the service worker code on that path. We will get an error like this
Screenshot 2019-12-23 at 3 00 35 PM
as we have disabled the parse request for that URL. This can be tested on a site I have created https://deepak-pwa.000webhostapp.com/

@pradeep910 pradeep910 added the enhancement New feature or request label Dec 24, 2019
@deepaklalwani97
Copy link
Member

@pradeep910 I have checked how other push services send push notifications and mostly all of them use a separate server for push notifications and do not store the user's subscription data in the user's database. And if agree with your point if a site has large number of subscribers say 50000 or something it will create a load on users site to avoid that we can use a separate notification server or some service like firebase to store subscription data and send push notifications.

CC: @chandrapatel

@pradeep910 pradeep910 changed the title [Idea] Push notification plugins support [PWA] Push notification plugins support Jan 21, 2020
@rtCamp rtCamp deleted a comment from deepaklalwani97 Jan 23, 2020
@rtCamp rtCamp deleted a comment from deepaklalwani97 Jan 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request planning Planning of a feature/task
Projects
None yet
Development

No branches or pull requests

2 participants