Skip to content

MAJ Canary - Destination Logs#277

Open
imroxxor wants to merge 19 commits intomainfrom
add-destination-logs
Open

MAJ Canary - Destination Logs#277
imroxxor wants to merge 19 commits intomainfrom
add-destination-logs

Conversation

@imroxxor
Copy link
Contributor

@imroxxor imroxxor commented Jan 23, 2026

🔍 Prévisualisation

Comment on lines +74 to +78
## Configuration des destinations

Chaque log peut être envoyé vers différentes destinations.

### Choix de la destination
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
## Configuration des destinations
Chaque log peut être envoyé vers différentes destinations.
### Choix de la destination
## Autre destination
Chaque type de log peut être configuré pour être envoyé vers un autre serveur ou une API externe.

Comment on lines +98 to +107
### Les options

::tabs
::tab{ label="Serveur" }
L'option par défaut, les logs sont envoyés dans un salon configuré du serveur.

> ***DraftBot** gère ses propres webhooks et l'envoi des messages avec ses derniers. Si vous supprimez son webhook de logs, il le recréera automatiquement lors de l'envoi du prochain log.*
::

::tab{ label="Autres serveurs" }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je propose une refonte de ce menu en titres et texte, on perd pas mal de contenu. Je te propose de décrire l'option serveur dans la description de "Configuration des destination" afin d'expliquer que c'est comme ça par défaut mais qu'on peut aussi l'envoyer ailleurs.

Ensuite on aurait 2 titres ### Autre serveur et API Externe l'un a la suite de l'autre, ça éviterait de faire penser qu'on a quasi rien dans la page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants