-
-
Notifications
You must be signed in to change notification settings - Fork 477
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"telegramEnabled" remains always true, even when trying to disable Telegram notifications (api) #4028
Comments
When you do a overseerr/server/routes/user/usersettings.ts Lines 264 to 284 in 3242251
When you go to overseerr/server/routes/user/usersettings.ts Lines 314 to 341 in 3242251
There is no toggle to enable/disable Telegram per user, except perhaps editing the Chat ID or set which notificationTypes for Telegram are sent to the user. |
Description
Hi Overseerr Team,
I've noticed that regardless of how I send the telegramEnabled field in the
POST /api/v1/user/<ID>/settings/notifications
endpoint, Overseerr always retains telegramEnabled: true.Setting it to false in the request body does not actually disable Telegram notifications. Instead notificationTypes.telegram is set to 0 (e.g., "telegram": 0)—at which point telegramEnabled still remains true, even though notifications are effectively disabled.
Not a big problem, but it just doesn't behave as you would expect.
I bypassed it by setting ‘notificationTypes’: {
‘telegram": 3661 } to activate and 0 to deactivate.
Version
1.33.2
Steps to Reproduce
Make a POST request to /api/v1/user/<OVERSEERR_USER_ID>/settings/notifications with:
Screenshots
Logs
Platform
desktop
Device
x86 PC AMD
Operating System
ubuntu 24.04 desktop
Browser
Firefox / Chrome
Additional Context
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: