Skip to content
This repository was archived by the owner on Jul 14, 2026. It is now read-only.

feat: create_post.py appelle l'API planning au lieu d'envoyer le webhook#28

Merged
mortifia merged 1 commit into
masterfrom
feat/create-post-use-api
Jul 12, 2026
Merged

feat: create_post.py appelle l'API planning au lieu d'envoyer le webhook#28
mortifia merged 1 commit into
masterfrom
feat/create-post-use-api

Conversation

@mortifia

@mortifia mortifia commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Réfère UnionRolistes/Bot_Base#79 (nécessite ce service pour fonctionner) et #26 ("move to bot_planing").

Le CGI envoyait directement le message Discord via le webhook fourni dans le formulaire, puis tentait (code commenté, jamais fini) de persister dans events.xml via un chemin relatif supposant un système de fichiers partagé avec le bot.

Remplacé par un simple appel POST /events vers la nouvelle API planning, qui se charge de l'envoi Discord (webhook enregistré par $jdr, retrouvé par user_id) et de la persistance. Plus besoin de discord.py/asyncio côté CGI.

Termine la partie Web_Planning de UnionRolistes/Bot_Planning_python#11.

Le CGI envoyait directement le message Discord via le webhook fourni dans
le formulaire, puis tentait (code commenté, jamais fini) de persister dans
events.xml via un chemin relatif supposant un système de fichiers partagé
avec le bot — incompatible avec des conteneurs séparés.

Remplacé par un simple appel POST /events vers la nouvelle API planning
(UnionRolistes/Bot_Base #api), qui se charge de l'envoi Discord (via le
webhook enregistré par $jdr, retrouvé par user_id) et de la persistance.
Plus besoin de discord.py/asyncio ni de connaître le webhook_url côté
formulaire : requests suffit.
@mortifia
mortifia merged commit 411e1c0 into master Jul 12, 2026
4 checks passed
@mortifia
mortifia deleted the feat/create-post-use-api branch July 12, 2026 04:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant