Add GetMessages and DeleteMessage to MessageHandler
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- go
- Domaine
- api, backend-api-design
Piste de recherche
Commencez par localiser l’interface MessageHandler et la structure Message, puis suivez leurs implémentations et l’utilisation de l’API dans tout le dépôt. Déterminez comment les identifiants de message doivent être représentés et quel doit être le comportement de l’énumération, de la récupération et de la suppression pour un plugin. Le travail est terminé lorsque l’interface et les implémentations prennent en charge les opérations demandées avec un comportement cohérent.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Currently, the MessageHandler has an API gap in that it cannot list, get, or delete messages. This would be valuable for plugins who want to display a persistent notification while some event is ongoing.
I think the MessageHandler interface could just be extended with a few extra functions
// MessageHandler consists of message callbacks to be used by plugins.
type MessageHandler interface {
// SendMessage sends a message with the given information in the request.
SendMessage(msg Message) error
// GetMessages returns all active messages sent by this plugin.
GetMessages() ([]Message, error)
// GetMessage returns a specific message by ID.
GetMessage(id uint) (*Message, error)
// DeleteMessages deletes all messages created by this plugin.
DeleteMessages() error
// DeleteMessage deletes a specific message by ID.
DeleteMessage(id uint) error
}
I think the Message struct might also need to be extended or a new struct would need to be defined that includes an ID.
As background, I want to make a plugin for Alertmanager where alerts will be persistently created as notifications in Gotify so that I only get one notification for the alert and it gets deleted automatically when it resolves. Additionally, I want to add special handling for the watchdog alert so if Alertmanager stops sending it, I get a notification. With the current plugin API, I am only able to send messages when I get them from alertmanager.
If this is a desirable inclusion, I would love to implement it.
Relates to #7
- Langage dominant
- Go
- Étoiles
- 16
- Forks
- 4
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de gotify/plugin-api
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
gotify/plugin-api#8 · 1 commentaire ·
-
Allow plugins to delete messages Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
gotify/plugin-api#7 · 1 réaction ·
-
Configurer comments not working. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
gotify/plugin-api#6 · 2 commentaires ·
-
new-api
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
gotify/plugin-api#3 · 4 commentaires · 1 réaction ·
Toutes les issues de gotify/plugin-api
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
bug group: validation priority: low
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
codecheckers/chekhov#51 ·
-
Creating worktree from an existing remote branch with a slash in it, has unexpected behaviour Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100