Introduction des créations automatique de salon

Virtual-Royaume / Royaume-Discord-Bot

It is the bot of our Discord community, it offers various utility features. It is written in TypeScript and uses DiscordJS to interact with the Discord API.

9 stars 1 forks source link

Introduction des créations automatique de salon #74

Closed Steellgold closed 2 years ago

Steellgold commented 2 years ago

Implémentation d'un systeme de création/suppression automatique des salons vocaux.

Bluzzi commented 2 years ago

Review faite, il y faudra aussi corriger le les points suivant :

[x] essayer de raccourcir le code d'avantage, je pense que les répétitions de code pour les salons privés pourrait être combiné avec le reste plus proprement
[x] corriger le bug de suppression du dernier salon crée lorsque tout les autres salons sont deja rempli

J'ai deja une idée de comment ceci pourrait être corrigé, donc je pourrais m'en occupé si tu le souhaite @Steellgold

Bluzzi commented 2 years ago

TODO pour la vérification des potentiels derniers bugs

[x] Tester la création des salons par défault (vérifier leur ordre aussi)
[x] tester la création de salon public/privé quand tout les salons actuels sont plein
[x] tester la suppression des salons et vérifier qu'elle ne spam pas comme avant

Bluzzi commented 2 years ago

Mon clean est fini @Steellgold, demain on test tout ça tout les deux puis si tout est bon cette PR pourra être merge !

Steellgold commented 2 years ago

Pas de problème, je suis disponible la journée comme d'habitude 👍

Steellgold commented 2 years ago

Il y a un bug, dès que le bot est chargé (- Info » Successfully registered application (/) commands), cela créer les salons BLEU, ROUGE, JAUNE, même si il existent

Steellgold commented 2 years ago

Par rapport aux salons public, le bot créer (correctement), mais des fois surpprime les salons de couleurs où il y a des personnes dedans

Steellgold commented 2 years ago

TODO pour la vérification des potentiels derniers bugs

[ ] Tester la création des salons par défault (vérifier leur ordre aussi)

[ ] tester la création de salon public/privé quand tout les salons actuels sont plein

[ ] tester la suppression des salons et vérifier qu'elle ne spam pas comme avant

[x] Tester la suppression lorsque des salons privés de haut dessus ne sont plus utilisés (Tests effectué avec @Squashhh)
- Exemple: Salon 1 est utilisé, donc ça créer Salon 2, une fois que Salon 1 est vide, ça supprime Salon 2 et ainsi de suite pour salon 3, 4 etc

Steellgold commented 2 years ago

Le salon privé (original) ce supprime, créer, supprime

https://user-images.githubusercontent.com/51505384/177796138-53ea25bf-bdc2-450c-990c-9f0dfc2e86fe.mp4

Steellgold commented 2 years ago

(Je voulais réouvrir la PR pour régler le bug)