Refactorisation (nettoyage de printemps)

[Aezurly]

Remarques sur la qualité de code

HTML & CSS

• Énormément de duplication de code dans des html css (ex. votes-grade.component et song-list)
• Utilisation abusive d'Angular Material et donc de !important
• Utilisation de <br> et autre margin hard coded (ex. )

UTILISATION DE ANGULAR

• Beaucoup de composants avec des noms pas clairs
• Trop de logique dans les composants
• Beaucoup d'enum et de constantes (et string magiques) qui traînent dans des composants
• Couplage élevé car passage des données avec des Input plutôt que des services

PS : Les points que j'aborde sont ceux qui vont rendre le maintien du code vraiment difficile si pas corrigé, j'ai pas regardé en détail l'utilisation de bonnes conventions etc.
En tant que chargé de projet 2 je vous le dis ce code est une catastrophe, il y a beaucoup de boulot 😬

[Aezurly]

Évidemment là c'est une issue assez globale il faudrait probablement redéfinir ça, je veux juste pas oublier les points que j'ai remarqué.

Je pourrais juste essayer de faire la refactorisation moi même mais j'ai juste un peu peur de briser de la logique sans m'en rendre compte puisqu'il n'y a pas de tests etc. Anyways là je me focus sur juste finir d'update à Angular 19 pour partir d'une base clean à jour puis il sera finalement temps de se mettre à nettoyer ce grenier.

[Ithyx]

Salut, et merci pour ton taf ! en effet, je pense qu'on est tous d'accord dans l'équipe pour dire que le code tel qu'il est est un piètre état, et ça fait maintenant quelques mois que Yatga/Kalimba (mais surtout yatga) sont en maintenance uniquement (donc pas ou très peu de nouvelles features). On réfléchis en interne sur la meilleure direction pour la suite du projet, mais on penche pour l'instant sur un repart de 0, avec un nouveau stack (on gardera probablement angular pour le frontend, mais on est en train de regarder comment changer de firebase vers un autre type de base de données). Pour l'instant, un autre projet, plus petit en taille, est notre focus, mais on veut s'occuper de ça d'ici pas trop trop longtemps.

Si ces discussions t'intéressent et tu veux t'intégrer plus au développement, je te conseillerais de ne pas trop passer de temps sur du refactoring et du clean-up de code qui de toute façon va devenir obsolète, mais j'ai déjà proposé à @hvdbm de t'offrir des responsabilité dans la prochaine refonte du site. On te tiens au courant, mais dis-nous d'abord si ça t'intéresserais, et si oui quel type de travail tu voudrais faire (on pensait front-end, mais selon une source anonyme (Victor), tu voudrais autre chose).