search

jazzband / django-push-notifications

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

Update webpush.py #604

Closed Fethienv closed 9 months ago

Fethienv commented 3 years ago

Add ttl keyword and get_wp_ttl

jamaalscarlett commented 9 months ago

This is unnecessary as the ttl can already be passed as a kwarg see: https://github.com/jazzband/django-push-notifications/blob/master/push_notifications/webpush.py#L40 and https://github.com/web-push-libs/pywebpush/blob/992efed89454629e741f8540f690bef681b17f2d/pywebpush/__init__.py#L377