search

remib18 / webtoon-like

Projet de second semestre de L1 informatique.
1 stars 1 forks source link

Options utilisateur (édition des infos, suppression de compte) #59

Closed g4bey closed 2 years ago

g4bey commented 2 years ago

Faudrait merge #56 d'abord. La branche est basée dessus.

g4bey commented 2 years ago

Beaucoup de répétitions d ne le css des pages qui aurait peut être pu être synthétisé. J'aurai aussi pensé à un seul handler pour toutes les modifications de comptes. Cependant, ce ne sont pas des problèmes à fixer car mineurs. Fix les deux trois autres problèmes relevés et ce sera déjà top !

petit changement dans les pages manager, je sais pas ce que t'en penses. Me suis dis que l'héritage est pertinent. Si y'a besoin d'overwrite pour une page, on peut.

g4bey commented 2 years ago

Désolé quelques oublies dans la dernière demande de modifications... Sinon j'aime bien ton idée mais est il réellement nécessaire d'avoir des classes qui étendent de messageManager ? Étant donné qu'elles font exactement la même chose ? Ps: C'est une question donc c'est à toi de voir hein. Si ça te vas comme ça moi aussi :)

En soit non, je vais les virer histoire de clarifier. J'imagine qu'il vaut mieux étendre si y en a vraiment besoin.

g4bey commented 2 years ago

Si pas d'erreurs lors de l'exécution, tout est bon 👍🏻

Normalement pas de pb. J'ai fait qques tests.