search

Pushwoosh / web-push-notifications

Web Push Notifications
Other
62 stars 28 forks source link

Lower the level of error to warning #33

Closed finnan444 closed 2 months ago

finnan444 commented 5 months ago

When I added pushwoosh script and enabled Firefox and Chrome integration it works fine but I don't want to enable it on Safari. So eventually in my error log system(sentry for example) I see a lot of errors of type Unhandled Promise Rejection: Error: For work with Safari Push Notification add safariWebsitePushID to initParams!

Screenshot ![Screenshot 2024-05-31 at 15 56 06](https://github.com/Pushwoosh/web-push-notifications/assets/28092100/bdc94b5b-5c9f-427e-8c87-284e02366dc9)

I understand that i can ignore this error in logs, but maybe better handle this promise rejection and show warning in console?

enginseer-dev commented 5 months ago

Hello @finnan444, Thank you for pointing this out! I'll pass your request to the corresponding team for further evaluation and let you know the result.

enginseer-dev commented 5 months ago

Hey @finnan444, I want to provide an update. The team will fix this soon, but I can't tell you any ETA yet. Thank you!

karudo commented 2 months ago

@finnan444 looks like the problem already fixed