Giter Site home page Giter Site logo

Comments (14)

lukasag avatar lukasag commented on September 6, 2024 6

I fixed it, it was all my mistake. The page at Google Developer Console has another menu on the top-left corner where I didn't noticed a button to change from the API Manager to the Developer Console (which I tough i was already in because of the page name). The whole thing is confusing, you are inside a page with a name that is not that actual page but is part of it. Or I need to drink some less coffee and get into this "sleep 8 hours a day" routine.

The link you sent was also redirecting me to the API page. Now that I got to it from the menu I said, when I go back to the API page it shows me a buch of other links in this same menu that were not displaying before... (wtf Google! Really, WTF!?)

from wp-web-push.

marco-c avatar marco-c commented on September 6, 2024

I've never seen this error, it could be somehow related to https://bugs.chromium.org/p/chromium/issues/detail?id=464365.

from wp-web-push.

marco-c avatar marco-c commented on September 6, 2024

Looks like you have two <link rel="manifest" in your page.
The plugin is adding <link rel="manifest" href="https://www.gameprosa.com.br/wp-content/uploads/wpservefile_files/manifest.json">.
Are you adding <link rel="manifest" href="/manifest.json?v=kPPJxwmvjB"> manually?

from wp-web-push.

lukasag avatar lukasag commented on September 6, 2024

I didnt noticed your plugin adds a manifest. I have my own manifest file with my icons and web app settings. How about an option to disable the plugin generated manifest? I'll try disabling in the plugin code to see if the problem gets solved, I'll reply again within the next hours.

from wp-web-push.

marco-c avatar marco-c commented on September 6, 2024

You just need to comment out the line $manifestGenerator = Mozilla\WebAppManifestGenerator::getInstance(); in wp-web-push-main.php.

If this fixes your problem, I'll add an option to disable the generation of the manifest.

from wp-web-push.

lukasag avatar lukasag commented on September 6, 2024

Marco, it did not fixed the problem.
Please check on https://www.gameprosa.com.br

Only one manifest file loading, still getting the "Uncaught (in promise) DOMException: Registration failed - push service error" error located at undefined, line 1. Unefined drives crazy, how can I properly debug this?

from wp-web-push.

marco-c avatar marco-c commented on September 6, 2024

The sender ID is usually a number, are you sure gameprosa-1285 is correct?

from wp-web-push.

marco-c avatar marco-c commented on September 6, 2024

image12

from wp-web-push.

lukasag avatar lukasag commented on September 6, 2024

My Dev. Console is different than yours. It does not have this content block and the overall design is pretty different. I've already tried to get to this same panel by going to this https://console.developers.google.com/home/dashboard URL but it always redirects me to https://console.developers.google.com/apis/
For this project the URL is https://console.developers.google.com/apis/library?project=gameprosa-1285
Even the page that shows all my projects is different and the only IDs it shows for my projects are like this one I am using.
Call me on Skype (lukasfire) if you want me to share my screen with you so you can see it, it's definitely weird.

from wp-web-push.

marco-c avatar marco-c commented on September 6, 2024

Try opening https://console.cloud.google.com/home/dashboard?project=gameprosa-1285&pli=1

from wp-web-push.

marco-c avatar marco-c commented on September 6, 2024

I filed google/WebFundamentals#2847 to update the Google documentation.

from wp-web-push.

lukasag avatar lukasag commented on September 6, 2024

Thanks Marco, sorry for wasting your time with this. The plugin is now working normally.
But the "enable manifest" button should be there because it will definitely cause problems to people who have their own manifest files. I also added a crossorigin="use-credentials" parameter to my manifest link.

from wp-web-push.

marco-c avatar marco-c commented on September 6, 2024

No worries, it was useful because I wouldn't have noticed that the documentation was outdated otherwise. Let's hope it's going to be fixed soon!

I'm going to file a new issue for the manifest thing.

from wp-web-push.

challenger532 avatar challenger532 commented on September 6, 2024

@lukasag hello, how did you solve it, I'm stuck with the same bug... wasted hours on that.. no result!
the same code is working on the browser, but when running with electron, I'm getting this bug...

from wp-web-push.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.