RocketChat/Rocket.Chat

Settings for confirmation for using @all notification on messages

Open

#909 geöffnet am 26. Sept. 2015

Auf GitHub ansehen
 (21 Kommentare) (1 Reaktion) (0 zugewiesene Personen)TypeScript (45.311 Stars) (13.573 Forks)batch import
area: ui/uxgood first issuegsoc-projecttriagedtype: improvement

Beschreibung

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.

Contributor Guide

Tech Stack
typescriptreactnodejs
Domain
frontendbackend
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
Familiarity with Rocket.Chat settingsBasic understanding of @all notifications
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30
Research-Richtung
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.