RocketChat/Rocket.Chat

Settings for confirmation for using @all notification on messages

Open

#909 ouverte le 26 sept. 2015

Voir sur GitHub
 (21 commentaires) (1 réaction) (0 assignés)TypeScript (45 311 stars) (13 573 forks)batch import
area: ui/uxgood first issuegsoc-projecttriagedtype: improvement

Description

Even though we had a few funny times when someone shoot an @all notification in the middle at the night on the Demo Server, what do you folks think about the idea of adding a settings-based confirmation before using it?

screen shot 2015-01-30 at 3 36 44 pm

Maybe it could be related to rules such as the number of users in the channel and/or number of different timezones etc.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Guide contributeur

Stack technique
typescriptreactnodejs
Domaine
frontendbackend
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Familiarity with Rocket.Chat settingsBasic understanding of @all notifications
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
Investigate how existing settings are implemented in Rocket.Chat (e.g., in the Settings module). Look at the codebase for @all notification handling, likely in message or notification services. Propose a UI for a confirmation dialog and a setting toggle. Consider the number of users/timezone rules as mentioned in the issue.