search

stephabauva / defi3-v2

0 stars 0 forks source link

Note défi 3 #1

Open lecascyril opened 2 years ago

lecascyril commented 2 years ago

Vidéo marrante, tout est là, c'est fonctionnel, y a les doc, les commentaires, et t'as reussi au final a bien deployer (avec du retard) , toujours bien d'utiliser bootstrap, c'est clean

J'ai juste eu une interrogation en parcourant ton projet, pourquoi avoir choisi le cas de draw?

D'un point de vue simplicité d'affichage, les boutons plutot que de les disable et d'avoir un front chargé pour pas grand chose, ne meme pas les afficher serait mieux

9,5/10

stephabauva commented 2 years ago

thx cyril, j'utilise juste TallyVotesDraw (renommée countVotes) parce que cette fonction met dans un array au minimum 1 gagnant, du coup je peux aussi l'utiliser en remplacement de TallyVotes. Du coup coté front je récupère la liste ( qui arrive en objet), et je juste prend de cette objet le ou les gagnants.