DraftBot-A-Discord-Adventure
Supprimer le modele et la table entities
Les colonnes maxHealth, speed, attack, defense ne sont d'ailleurs plus utilisées
- [ ] quand on récupère un player via getPlayerbyid on n'obitent un objet qui ne peut pas être utilisé, par exemple, pour faire un player.getPseudo
du coup j'update le commentaire ci-dessus vis à vis d'une modif sur la table player qui va avec
de plus, faire en sorte de rajouter dans le modèle de player un attribut User, qu'on attribue hors database, et qui serait init par défaut lors du passage en paramètre de l'entity dans executeCommand. Il serait donc plus aisé d'utiliser les Users des autres entities, et ce sera plus la peine de rajouter des types UserInformations partout où c'est nécessaire
de plus, faire en sorte de rajouter dans le modèle de player un attribut User, qu'on attribue hors database, et qui serait init par défaut lors du passage en paramètre de l'entity dans executeCommand. Il serait donc plus aisé d'utiliser les Users des autres entities, et ce sera plus la peine de rajouter des types UserInformations partout où c'est nécessaire
Je suis contre, après on aura des player avec user null ça va être le bordel
Et aussi vaut mieux garder au maximum discord.js en dehors des modèles ou du code plus générique
au vue de ce que @Feiryn a dit, je suis d'accord, donc, sauf info contraire je crois que cette issue est finie
faudrait confirmer que les deux points de la description ont bien été fait, ils sont légitimes mais le titre de l'issue est fait