Skip to content

Commit

Permalink
Add WebPush support for Safari (#674)
Browse files Browse the repository at this point in the history
* Add WebPush support for Safari

* Update webpush.py based on review

Declare results variable at the start of the block

* Fix typo in warning

Co-authored-by: Cuyler Stuwe <salembeats@gmail.com>

* Update README.rst

Co-authored-by: James Bligh <618250+blighj@users.noreply.github.com>

* Fix mailto: space

* Expanded documentation for Web Push (#558)

• Added example code to register WP device
• Fixed issue where call to extract UserAgent didn't include UA
• Added examples on how to send a Web Push message

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Attempt to fix tests

* Update README.rst

---------

Co-authored-by: James Bligh <james.bligh@silvercloudhealth.com>
Co-authored-by: Cuyler Stuwe <salembeats@gmail.com>
Co-authored-by: Éloi Rivard <eloi.rivard@nubla.fr>
Co-authored-by: Neil Littlejohns <n.littlejohns@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
  • Loading branch information
6 people authored and 50-Course committed Oct 3, 2024
1 parent 5f8d2bd commit 39f8c17
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ setup_requires =
APNS =
apns2>=0.3.0
importlib-metadata;python_version < "3.8"
pywebpush>=1.3.0
Django>=2.2

WP = pywebpush>=1.3.0
Expand Down
2 changes: 1 addition & 1 deletion tests/test_rest_framework.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ def test_validation(self):

# valid data - 200 bytes mixed case
serializer = APNSDeviceSerializer(data={
"registration_id": "aE" * 200,
"registration_id": "aE" * 100,
"name": "Apple iPhone 6+",
"device_id": "ffffffffffffffffffffffffffffffff",
})
Expand Down

0 comments on commit 39f8c17

Please sign in to comment.