muraliwebworld / push-notification-plugin-for-wordpress-websites

Push Notification WordPress plugin in php, javascript using Firebase httpv1 for websites, Android/IOS mobile apps with PWA progressive web app
https://wordpress.org/plugins/push-notification-for-post-and-buddypress/
GNU General Public License v3.0
13 stars 1 forks source link

undefined issue causing failed page loads #17

Open itzteajay-glitch opened 5 months ago

itzteajay-glitch commented 5 months ago

So I've ran into an issue with the onesignal configuration where this plugin makes the frontend throw the below error. When I uninstall the plugin the page loads fine but I'm really hoping to use this plugin since it integrates so nicely w/ buddypress. I was having issues with push notifications coming through using the firebase process but I may have to fall back to that until this issue is resolved.

Screenshot 2024-06-03 124229

muraliwebworld commented 5 months ago

I will look into it and update you as early as possible

muraliwebworld commented 5 months ago

Elementor compatiabilty is already resolved in 1.81 version itself, it was caused due to variable named safari and it was resolved in 1.81 version itself now plugin version 1.88, i verified in the code there is no safari variable in the plugin and i updated code in 1.81 version itself not to use standard variable safari. Could you please let me know which version of plugin are you using? Following is link in WordPress forum regarding this, https://wordpress.org/support/topic/elementor-compatibility-and-duplicated-notifications/

itzteajay-glitch commented 5 months ago

So looking at the plugins page I'm currently only version 1.87 but I do see there is an update to the plugin this morning. Falling back to firebase is working for me ATM so I'm not overly concerned but I thought I'd bring it to your attention.

On Wed, Jun 5, 2024, 12:24 AM Muralidharan Ramasamy < @.***> wrote:

Elementor compatiabilty is already resolved in 1.81 version itself, it was caused due to variable named safari and it was resolved in 1.81 version itself now plugin version 1.88, i verified in the code there is no safari variable in the plugin and i updated code in 1.81 version itself not to use standard variable safari. Could you please let me know which version of plugin are you using? Following is link in WordPress forum regarding this,

https://wordpress.org/support/topic/elementor-compatibility-and-duplicated-notifications/

— Reply to this email directly, view it on GitHub https://github.com/muraliwebworld/push-notification-plugin-for-wordpress-websites/issues/17#issuecomment-2149068522, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANHRLN3RNXXN37I2SVG5U4LZF24J3AVCNFSM6AAAAABIXEK5L6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBZGA3DQNJSGI . You are receiving this because you authored the thread.Message ID: <muraliwebworld/push-notification-plugin-for-wordpress-websites/issues/17/2149068522 @github.com>