search

jazzband / django-push-notifications

Send push notifications to mobile devices through GCM or APNS in Django.
MIT License
2.23k stars 605 forks source link

Added Safari browser to web-push #687

Closed Sergey-Yalovoy closed 9 months ago

Sergey-Yalovoy commented 11 months ago

I have studied the problems with Safari and offer my solution. I tested my solution in my project, it works great.

jamaalscarlett commented 10 months ago

Looks good @Sergey-Yalovoy . If you rebase against master, I have fixed the failing test actions.

jamaalscarlett commented 10 months ago

Will this require a new migration? See: https://github.com/jazzband/django-push-notifications/blob/master/push_notifications/migrations/0008_webpush_add_edge.py

azmeuk commented 10 months ago

As I understand, with #674 makes WP browser generic, so this PR won't be needed.

jamaalscarlett commented 9 months ago

Closing as this is addressed by https://github.com/jazzband/django-push-notifications/pull/674