RocketChat / Rocket.Chat

The communications platform that puts data protection first.
https://rocket.chat/
Other
39.96k stars 10.3k forks source link

Prompt user to disable offline email notifications when his mobile push notifications are enabled #2839

Open ulope opened 8 years ago

ulope commented 8 years ago

Your Rocket.Chat version: 0.25.0

It would be great if the "offline email notifications" would only be sent if (mobile) push notifications are not getting through to the user.

As it is now I get two notifications for every message on my smartphone: one push from RC and one email.

jclabaugh commented 8 years ago

Could this be configurable?

engelgabriel commented 8 years ago

@This is configurable on My Account -> Preferences

image

danpospisil commented 7 years ago

Any update on this? We would appreciate this feature a lot.

gdelavald commented 7 years ago

Hey, looking into this issue. Would this be configured in a room-to-room base in the room notification settings? I didn't find any Push Notification specific field on the Account Settings.

karlprieb commented 7 years ago

Hey @gdelavald I think it could be a user setting on "Account Settings" 👍

gdelavald commented 7 years ago

If I add a new user setting for Global Push Notification, this would probably reflect in the whole process of sending notifications (same way email does, checking the global configs and then the room configs). Also, would I need to create a new option on the Push Notifications for each channel, like the Audio/Email do ("Use Account Preference")?

gdelavald commented 7 years ago

I guess not, looks like #4339 already solves my concerns.

gdelavald commented 7 years ago

If I understand correctly the point of this ticket is to create something like this:

avoid flood message

This would be triggered whenever the user changed the Push Mobile and Email Notifications to something different than nothing/disabled, which would cause some cases where the user gets notified twice.

gdelavald commented 7 years ago

The changes here are based on @alexbrazier #4339 PR for notifications options for user settings.

ritwizsinha commented 4 years ago

I would like to work on it

Akshanshkaushal commented 2 months ago

Hi @engelgabriel and the Rocket.Chat team - I'm interested in working this issue!