Problem/Motivation

When I want to use some custom provider of push notifications together with (or separate) PWA provider from then Social PWA module then I can not create some settings wrapper on the user settings page.

Proposed resolution

Create a new module which provides a new plugin for creating instances for each provider of push notifications as PWA, Firebase, etc.

Comments

chmez created an issue. See original summary.

ohorbatiuk’s picture

Status: Active » Needs work
ribel’s picture

Status: Needs work » Needs review

The idea behind this refactoring and code looks good to me.
Just we need to be sure that it is merged and released together with https://github.com/goalgorilla/social_pwa/pull/9
And please note that we update Message module to stable version within this PR.

ohorbatiuk’s picture

Version: 8.x-7.x-dev » 8.x-8.x-dev
ohorbatiuk’s picture

Assigned: ohorbatiuk » Unassigned
ronaldtebrake’s picture

Status: Needs review » Fixed

Will be in 7.2!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.