search

Blagues-API / blagues-api

😂 API de Blagues française et Open Source
https://www.blagues-api.fr
MIT License
46 stars 17 forks source link

Dispatcher dynamique #428

Closed ghost closed 2 years ago

ghost commented 2 years ago

J'avais eu l'idée de rendre la fonction "register" de la classe "Dispatcher" dynamique pour charger automatiquement les fichiers des commandes... je n'ai aucune idée de comment faire ça, mais dans l'idée ça pourrais être plus propre et plus pratique de faire comme ça, que de devoir continuer à lister toutes les commandes dans le dispatcher…

DraftProducts commented 2 years ago

Au vu du nombre de commandes qu'on a je sais pas si ça vaut le coup, mais si on valide le fait de le faire, je suis d'accord de m'en occuper 👍

achille-bsc commented 2 years ago

Ouais, c'est vrai qu'on n'a pas beaucoup de commandes... mais à force d'en ajouter ça va finir par faire un certain nombre... et puis je trouve ça toujours un peu plus propre que de devoir tous les citer un à un non ? (après, je dit ça, mais je ne sais pas faire ce genre de handlers... donc si ça se trouve, ce n'est pas du tout le cas...)

DraftProducts commented 2 years ago

C'est vrai que t'as raison, c'est plus propre.

DraftProducts commented 2 years ago

Je m'en occuperais 👍

achille-bsc commented 2 years ago

Je m'en occuperais 👍

Cool ! j'ai hâte de voir comment tu feras ! ça m'intéresse à fond !