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

OneSignal Push Notifications not working! #1404

Closed
anirudhpulikonda opened this issue Nov 12, 2017 · 12 comments
Closed

OneSignal Push Notifications not working! #1404

anirudhpulikonda opened this issue Nov 12, 2017 · 12 comments
Assignees
Milestone

Comments

@anirudhpulikonda
Copy link

I've enabled push notifications and configured with right APP ID when it first released in 0.9.67

But unfortunately, I'm not seeing any popup to subscribe to notifications. Even in 0.9.68 update also, I'm not getting any popup.

Can you guys look into this? Thanks in advance!

@MohammedKaludi MohammedKaludi added this to the 0.9.69 milestone Nov 14, 2017
@MohammedKaludi
Copy link
Collaborator

Hey @anirudhpulikonda,

We have fixed some issues that were seen for push notifications in the last update.
Can you please share the url of the site where you are adding?

Thanks,
Mohammed Kaludi

@anirudhpulikonda
Copy link
Author

Hey @MohammedKaludi

URL: https://www.watchmetech.com/

After 0.9.69 update also I'm not seeing any popup to the user which lets him subscribe to Onepush. I'm also seeing a weird URL popping up in my analytics whenever my readers open my blog's amp pages.

@MohammedKaludi
Copy link
Collaborator

MohammedKaludi commented Nov 14, 2017

This is strange, I can see the code is being added by our plugin http://take.ms/UK8SB and http://take.ms/55XIJ

But not sure why it is not triggering?

Thank you,
Mohammed kaludi

@anirudhpulikonda
Copy link
Author

@MohammedKaludi I'm sharing the credentials via the email above-mentioned.

Please let me know the changes you've done after fixing the issue.

Also, can you look into one more issue I was facing for many months with amp for wp?

My server supports New relic tracking, but all others WordPress sites on my server removes new relic script when ampforwp is used. But, the current site which we're discussing doesn't bypass the new relic script and I'm facing validations issues.

@MARQAS
Copy link
Contributor

MARQAS commented Nov 15, 2017

@anirudhpulikonda Can you share the URLs of where you are having validation issues?

@anirudhpulikonda
Copy link
Author

@MARQAS the same site which we're discussing here. URL: https://www.watchmetech.com/

I've sent the credentials via email. As of now, I've disabled the New Relic on my server, because this application is not able to disable the New Relic script when it's enabled.

So, If you can also look into this issue, I'll be very thankful. Please, let me know after fixing Onesignal issue so that I can enable New Relic in order check the issue.

MARQAS added a commit that referenced this issue Nov 15, 2017
@VizeyeBasvur
Copy link

Please inform in case any progress achieved. I have the same issue, Redirect plugin sees many of my some amp links that contains onesignal app id as 404.

MARQAS added a commit that referenced this issue Nov 16, 2017
MARQAS added a commit that referenced this issue Nov 16, 2017
MARQAS added a commit that referenced this issue Nov 16, 2017
@MARQAS
Copy link
Contributor

MARQAS commented Nov 16, 2017

It is working properly now.
@anirudhpulikonda and @VizeyeBasvur You guys can clone this branch or can wait for Saturdays update.

@MARQAS
Copy link
Contributor

MARQAS commented Nov 16, 2017

Thank you so much for reporting this and supporting us

@anirudhpulikonda
Copy link
Author

Thanks @MARQAS for looking into the issue, I'll wait for the update. BTW, my New Relic Script issue on AMP also fixed, It was Glue by Yoast which is causing the trouble. Once, Again thanks a ton for fixing issue with Advanced AMP Ads here: #1379

@Zabi09
Copy link
Contributor

Zabi09 commented Nov 17, 2017

@MohammedKaludi
Copy link
Collaborator

Push notification is working properly and above fix is tested on multiple sites.
Closing the ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants