Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor global #141

Open
4 of 12 tasks
0xN0x opened this issue Mar 22, 2024 · 3 comments · Fixed by #143
Open
4 of 12 tasks

Refactor global #141

0xN0x opened this issue Mar 22, 2024 · 3 comments · Fixed by #143
Labels

Comments

@0xN0x
Copy link
Contributor

0xN0x commented Mar 22, 2024

Ce ticket servira de liste des futurs refactor à faire, elle est non exhaustive et pourra évoluer par la suite :

  • Passage du linter sur tout le bot (Passage du linter sur tout le code #143)
  • Déplacement de toutes les sous commandes dans des fichiers indépendants (Split subcommands into multiples files #127)
  • Nettoyer les fonctions (utils entre autres) pour rendre les fichiers plus cohérents et avoir moins de code spaghetti
  • Arrêt de l'utilisation de collecteur (quand possible) pour passer sur les events interactionCreate et messageReactionAdd (au minimum pour les réactions) et pouvoir diviser ces evenements en sous fichiers pour améliorer la lisibilité et la cohérence du code
  • Remplacement de l'utilisation des réactions par des interractions (boutons, modals, selects, etc.)
  • Arrêt complet du support de commande non-slash (remove prefix and retryForever #128)
  • Remplacement des has et get par fetch pour les requête dans le cache de dJS
  • Nettoyage du code pour les warnings et erreurs (Refactor errors and badpractices #144)
  • Ajout de nouveaux commentaire et correction de ceux déjà existants
  • Passage sur tous les TODOs pour les retirer si plus a jour ou les réaliser (ou faire un ticket si trop gros)
  • Réduction des intéractions en MP le plus possible (cf. Éviter au max les MP #75)
  • Suppression de code mort (cf. Gestion interdiction sur user blacklisté #28)

On pourra linker les PR à chaque éléments lorsqu'ils seront fait. Tout n'est pas à la même priorité mais au moins ça nous permettra d'avoir une liste non exhaustive de ce qu'il reste à faire pour la partie refactor.

@rick-gnous
Copy link
Member

Normalement la dernière commande non slash a sauté dans 0e83829

@0xN0x
Copy link
Contributor Author

0xN0x commented Mar 22, 2024

Normalement la dernière commande non slash a sauté dans 0e83829

D'acc j'ai update pour mettre la PR, j'avais pas tout suivi la dessus

@rick-gnous rick-gnous linked a pull request Mar 26, 2024 that will close this issue
@rick-gnous
Copy link
Member

Il faudrait mettre un ephemeral lorsque l'utilisateur n'est pas enregistré (libre à vous de modifier ce message pour rajouter d'autres cas, on en fera une liste)

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 a pull request may close this issue.

2 participants