overseerr icon indicating copy to clipboard operation
overseerr copied to clipboard
verified publisher icon sct

fix: delete endpoint on push notification disable

Open OwsleyJr opened this issue 6 months ago • 1 comments

Description

Quick PR that deletes the current endpoint on the users device when disabling the push subscription. Added a check on the backend in case the endpoint doesn't match to prevent any errors (it's possible there is a scenario where the endpoint has changed due to it being refreshed).

  • Will prevent rare scenarios where user can enable/disable a few times and load the DB

Screenshot (if UI-related)

N/A

To-Dos

  • [x] Successful build yarn build

OwsleyJr avatar Aug 15 '25 03:08 OwsleyJr

overseerr    Run #2874

Run Properties:  status check passed Passed #2874  •  git commit 805d539619: fix: delete endpoint on push notification disable
Project overseerr
Branch Review refs/pull/4222/merge
Run status status check passed Passed #2874
Run duration 02m 17s
Commit git commit 805d539619: fix: delete endpoint on push notification disable
Committer Brandon Cohen
View all properties for this run ↗︎
Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 28
View all changes introduced in this branch ↗︎

cypress[bot] avatar Aug 15 '25 03:08 cypress[bot]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Dec 17 '25 23:12 stale[bot]