Merci pour le code review. Je vais pouvoir améliorer mon code avec les conseils que tu m as donner. 👍 ça me boost a persévérer malgré les difficultés à trouver un job
Bonjour avoir regarder cette revu de code. Cela me permet de comprendre les axes de progrès a suivre. Certes je suis sorti d'un bootcamp, mais jamais le formateur est revenu sur un code. Nous montrer les bonnes pratiques. Certes je suis passé seul a coder avec typescript, aujourd'hui j'apprends spring boot et Angular. Mais cette vidéo me fait prendre conscience que j'ai du chemin encore. Non pas dans la connaissance de React, mais de perfectionner mon aborescence et cette vidéo m'a montré quelques points que j'ai vu sur Angular. Enfin merci de cette video.
Merci Joanny pour ton long message 😲Il m'a fait grave plaisir :)) Ça me rend heureux de voir que mes vidéos aident sincèrement les gens à gagner du level :)
@@videv9858 Heureux si cela a fait ton dimanche ☺. Je me suis abonné et je lâcherai un petit mot sur Linkedin parmi les quelques juniors que je côtoie si cela peut faire monter ta chaine...
Excellente vidéo, on ne trouve pas beaucoup ce genre de concepts de review qui peut aider à faire progresser les apprenants en insistant sur les bonnes pratiques.
j'ai bcp aimer regarder les projets et ton intervention est vraiment interessante j'espere que j'aurais le temps de m'ameliorer et te soumettre un de mes projets
Bonjour Vi, Des commentaires m'aident beaucoup. Étant autodidacte, c'est parfois dur de jauger son niveau et c'est super. J'espère envoyer un projet sous peu pour la review. Et si tu peux faire une vidéo sur l'architecture des projets TypeScript + React ou même React seul, c'est intéressant également
c'est une video vraiment super intéressantes ! il est possible dans refaire une du même styles ? je suis même près a me mettre sur un projet pour si c'est ok
@videv9858 a super ça ! C'est une très bonne nouvelle car j'ai trouver ça super intéressant ! Si c'est possible oui j'aimerais beaucoup ! Il est pas terminé mais si j'ai un délai de 3 voir grand max 4 semaines il sera clairement terminé oui ! Si c'est ok je me focus dessus 🔥
@@AzmirTefa en description de la vidéo, t'as un lien auquel tu peux envoyer le lien du repo sur git (attention, pas le lien de ton compte github seulement, sinon je ne vais pas savoir quel repo regarder). Envoie le moi mnt, ou dans 4 semaines. C'est pas certain que je regarde le tiens car si il n'est pas assez "intéressant pour les autres" je vais préférer en privilégier un autre. Donc avance sur ton projet pour toi, pas pour moi.
Bonjour Vi merci pour ce live très instructif, j'avais une question concernant l'employabilité d'un candidat ,est-ce que l'âge est-il un frein pour trouver un emploi dans la tech surtout si celui-ci vient d'une reconversion professionnelle et aurrait les bonnes pratiques professionnelles comme celles de ta formation ? 😊
@@yanb4315 L'âge ça peut jouer mais si t'es suffisamment bon et que tu coches toutes les cases, l'âge ils s'en moquent complètement tellement on a du mal à trouver des bons devs.
Le fait d'avoir que 3 commit me choque pas. Dans mes projets, la config par defaut du git c'est - delete branch on merge - squash Donc pour une branche avec 200 commits, elle sera squash en un seul quand on merge et la branch originale est supprimee. Ca evite d'avoir un master incomprehensible ou les commits intermediaires de plusieurs branches sont entremeles, avec des commit a la con genre fix, fix, fix, fix... C'est d'autant plus important que quand tu dois rollback une feature, si tu dois retrouver les 200 commits qui lui correspondent, t'es dans la merde. Quand c'est squash dans un seul, t'as qu'un commit a revert. Apres, avec mes devs, je suis un nazi sur l'historique, meme dans les branches. Si tu request une review sur une branche ou ya pleins de commits "fix fix fix", tu te prends une mandale. C'est ok d'avoir des commit temporaires quand tu cherches a resoudre un bug (particulierement quand t'arrives a reproduire qu'en staging) mais quand le bug est resolu, un coup de rebase interactive pour merge tous les commit foireux avec un vrai nom. Pareil, les commits "fix specs", c'est taquet. Tu trouves le commit qui pete les specs et tu l'edit. Si quelqu'un se demande pourquoi, la raison est simple. Allez review une branche avec 2 semaines de dev dedans (donc des milliers de modifs) sans y aller commit par commit. C'est l'enfer. 38:30 "bonnes technos => redux" en 2024??? Genre useReducer? non?
En soi, tu as raison. Chez nous aussi on git squash les commits avant de merger. Mais là, c'était dans un but pédagogique et de montrer au lead dev que le candidat (ici, Phillipe) savait utiliser git. Et l'histoire m'a donné raison : le dev en question (i.e. Phillipe) ne faisait pas beaucoup de commits, pas par choix, mais par oubli ou parce qu'il n'est pas à l'aise avec git de manière générale. Redux Toolkit* monsieur, ça change tout. Et check du côté software craftmanship, redux et redux toolkit sont très utilisés (si ce n'est, "la" solution de référence). Au dela de ça, c'est une "bonne techno" surtout parce que ça recrute bien sur du redux. Check les offres d'emploi, c'est pas toujours React Query ou Zustand ou autre qui sont demandés. Redux est encore très souvent demandé. Et useReducer, ça sert à rien. C'est hyper couplé à React et c'est juste une pâle copie de Redux mais dans le monde de React en moins bien. Autant prendre le vrai Redux. "Oui mais gneu gneu installer Redux , c'est lourd, tu installes une lib pour rien" ==> l'optimisation des packages, c'est pas l'enjeu ici. L'enjeu c'est de décrocher un job après une reconversion. Donc si faut installer redux (toolkit) pour montrer qu'on sait l'utiliser, j'encourage à le faire. C'est pas les opti sur les tailles de package qui vont payer les factures.
@@videv9858 Je dois confesser je connais pas du tout l'etat du marche du dev FE en france (puisque jy vis pas) Apres, on va pas lancer un debat sur l'opti des tailles de package, c'est javascript, je vais devenir maboule en 2s tellement la gestion des dependences de ce langage me sort par tous les orifices ;)
Pour la partie git je suis pas tout à fait d'accord, perso quand je monte un petit projet avant la v1 je commit pas mal de "wip" etc et quand je sors la v1 je repars avec un repo clean depuis une branche orpheline, si le projet n'évolue pas, y a pas de commit.
je te partage des bonnes pratiques pour les projets en entreprise qui eux, évoluent toujours. Mais chacun a sa manière de faire. Là, tu vois la mienne :)
Oui , frère vi perso j'aimerais revoir plus souvent ce genre de format et celui d'avant usage de usehook...et tous ce qui est concepte un peu flou sur reatct ou JavaScript ou même typescript
C'est pas un peu overkill de remplacer le button par Button? Ca prend la même place, mais on a un fichier en plus. Je vois pas trop l'intérêt dans ce cas précis.
L'intérêt c'est la réusabilité du composant. Et son plus gros avantage c'est que tu codes une fois et tu le réutilises plusieurs fois. Et si tu as besoin de changer le style de tous les buttons, t'as juste besoin de le changer à un seul endroit (dans Button.jsx) et non dans toouuuuuus les qu'il y a dans ton projet.
C'est prevu. Pas pour cette fois malheureusement mais j'espere la prochaine session. Ceci dit, je serais curieux de connaitre la raison de ton commentaire, t'inquiete je ne suis pas susceptible, j'accepte toutes les critiques constructives. D'ailleurs depuis ce live, j'ai deja adopté de nouvelles pratiques comme par exemple l'utilisation systematique de git, et la refactorisation systematique en composants reutilisables. Par contre je n'ai pas encore switché sur Typescript, un gros morceau pour moi que je prefere laisser pour plus tard
@@jovialiste hey salut ! Pour te dire je trouve que tu as une bonne approche pour t’organiser. Il te manque juste les best practices (que j’ai appris avec Vi à l’époque). Dans sa formation on est sur 15 tickets et du ticket 7 au 15 on apprend beaucoup à organiser le code de façon claire et modulaire. Pour moi si tu fais son projet, tu auras pas de mal à créer tes propres projets sur la base de sa structure. Après j’avoue que son pack vidéo est à 700€ et c’est pas donné non plus donc ça se comprend .. mais moi lorsque commencé, je me suis dis qu’un plombier doit bien acheter du matériel avant de travailler.. donc ça fait un peu parti du matériel 😅😅 Sinon bravo ! Ton projet est bien codé, il te faut progresser en UI-UX mais à la limite c’est pas grave car en entreprise on a souvent les maquettes du designer donc tu as juste à connaître figma .. 😊 En tout cas continue ! Ne lâches pas le dev il manque plus grand chose 🏋️♂️
Merci pour le code review. Je vais pouvoir améliorer mon code avec les conseils que tu m as donner. 👍 ça me boost a persévérer malgré les difficultés à trouver un job
Yes, courage ! T'es pas loin du niveau qu'on attend !
Il est beau le projet, j’aime beaucoup les couleur de l’interface !
Écrire Accueil et non Acceuil dans le haeder 😊
Super concept ! Et merci pour la review !
Avec plaisir monsieur ! 🎩
Bonjour avoir regarder cette revu de code. Cela me permet de comprendre les axes de progrès a suivre. Certes je suis sorti d'un bootcamp, mais jamais le formateur est revenu sur un code. Nous montrer les bonnes pratiques. Certes je suis passé seul a coder avec typescript, aujourd'hui j'apprends spring boot et Angular. Mais cette vidéo me fait prendre conscience que j'ai du chemin encore. Non pas dans la connaissance de React, mais de perfectionner mon aborescence et cette vidéo m'a montré quelques points que j'ai vu sur Angular. Enfin merci de cette video.
Merci Joanny pour ton long message 😲Il m'a fait grave plaisir :)) Ça me rend heureux de voir que mes vidéos aident sincèrement les gens à gagner du level :)
@@videv9858 Heureux si cela a fait ton dimanche ☺. Je me suis abonné et je lâcherai un petit mot sur Linkedin parmi les quelques juniors que je côtoie si cela peut faire monter ta chaine...
Mec il nous faut plus de video comme sa, sa nous explique bien react et ta manière d'expliquer est tres claire !
Merci merci ! :))
Excellente vidéo, on ne trouve pas beaucoup ce genre de concepts de review qui peut aider à faire progresser les apprenants en insistant sur les bonnes pratiques.
Merci Nico ❤
Merci Vi pour la review !!
Rare de voir ce genre de vidéo car en te regardant coder on apprend pas mal 🎉
Avec plaisir, c'est le but ! J'en referrai d'autres 😊
Video et concept précurseur !
Apprendre par la correction d’un projet niveau débutant voir intermédiaire c’est excellent !
Merci Vi t’es le meilleur 🙏🏽
Merci Hervé :)
j'ai bcp aimer regarder les projets et ton intervention est vraiment interessante
j'espere que j'aurais le temps de m'ameliorer et te soumettre un de mes projets
Avec plaisir, je ferai une nouvelle session bientôt :)
Hyper intéressant comme analyse de projets. Merçi
Franchement un collègue comme toi ça doit être le feu !!!
Merci Dypser, super sympa ton commentaire ! 😃
Merci pour la vidéo 👍
Merci pour ton commentaire 🙏
Bonjour Vi,
Des commentaires m'aident beaucoup.
Étant autodidacte, c'est parfois dur de jauger son niveau et c'est super.
J'espère envoyer un projet sous peu pour la review.
Et si tu peux faire une vidéo sur l'architecture des projets TypeScript + React ou même React seul, c'est intéressant également
Merci pour ton commentaire Chad et pour la vidéo, c'est noté 👀✍
Super live ! ❤
@@PikminPiPi merci petit pikminoux
Super le codeReview je me suis retrouvé et j'acquis les bonne pratique
Autodidacte depuis le Benin
vi Merci beaucoup
c'est une video vraiment super intéressantes ! il est possible dans refaire une du même styles ? je suis même près a me mettre sur un projet pour si c'est ok
Fin du mois ouais y'a moyen que j'en refasse une 😉 T'as un projet à toi que tu veux que je review ? 👀
@videv9858 a super ça ! C'est une très bonne nouvelle car j'ai trouver ça super intéressant ! Si c'est possible oui j'aimerais beaucoup ! Il est pas terminé mais si j'ai un délai de 3 voir grand max 4 semaines il sera clairement terminé oui ! Si c'est ok je me focus dessus 🔥
@@AzmirTefa en description de la vidéo, t'as un lien auquel tu peux envoyer le lien du repo sur git (attention, pas le lien de ton compte github seulement, sinon je ne vais pas savoir quel repo regarder). Envoie le moi mnt, ou dans 4 semaines. C'est pas certain que je regarde le tiens car si il n'est pas assez "intéressant pour les autres" je vais préférer en privilégier un autre. Donc avance sur ton projet pour toi, pas pour moi.
Concept très enrichissant. J'adore !
Merci Anthony :)
1:12:00 Ce n'est pas une faille de sécurité de stocker le token "en clair" dans le state ? N'est ce pas préférable de stocker le token hashé ?
Bonjour Vi merci pour ce live très instructif, j'avais une question concernant l'employabilité d'un candidat ,est-ce que l'âge est-il un frein pour trouver un emploi dans la tech surtout si celui-ci vient d'une reconversion professionnelle et aurrait les bonnes pratiques professionnelles comme celles de ta formation ? 😊
La personne la plus âgée que je connais et qui a réussi à trouver un job après reconversion avait 42 ans. Après, je connais pas.
@@videv9858 merci pour ta réponse !
@@videv9858 Mince j'en suis à déjà à 49 😅
@@yanb4315 L'âge ça peut jouer mais si t'es suffisamment bon et que tu coches toutes les cases, l'âge ils s'en moquent complètement tellement on a du mal à trouver des bons devs.
Le fait d'avoir que 3 commit me choque pas.
Dans mes projets, la config par defaut du git c'est
- delete branch on merge
- squash
Donc pour une branche avec 200 commits, elle sera squash en un seul quand on merge et la branch originale est supprimee. Ca evite d'avoir un master incomprehensible ou les commits intermediaires de plusieurs branches sont entremeles, avec des commit a la con genre fix, fix, fix, fix... C'est d'autant plus important que quand tu dois rollback une feature, si tu dois retrouver les 200 commits qui lui correspondent, t'es dans la merde. Quand c'est squash dans un seul, t'as qu'un commit a revert.
Apres, avec mes devs, je suis un nazi sur l'historique, meme dans les branches. Si tu request une review sur une branche ou ya pleins de commits "fix fix fix", tu te prends une mandale. C'est ok d'avoir des commit temporaires quand tu cherches a resoudre un bug (particulierement quand t'arrives a reproduire qu'en staging) mais quand le bug est resolu, un coup de rebase interactive pour merge tous les commit foireux avec un vrai nom.
Pareil, les commits "fix specs", c'est taquet. Tu trouves le commit qui pete les specs et tu l'edit.
Si quelqu'un se demande pourquoi, la raison est simple. Allez review une branche avec 2 semaines de dev dedans (donc des milliers de modifs) sans y aller commit par commit. C'est l'enfer.
38:30 "bonnes technos => redux" en 2024??? Genre useReducer? non?
En soi, tu as raison. Chez nous aussi on git squash les commits avant de merger. Mais là, c'était dans un but pédagogique et de montrer au lead dev que le candidat (ici, Phillipe) savait utiliser git. Et l'histoire m'a donné raison : le dev en question (i.e. Phillipe) ne faisait pas beaucoup de commits, pas par choix, mais par oubli ou parce qu'il n'est pas à l'aise avec git de manière générale.
Redux Toolkit* monsieur, ça change tout. Et check du côté software craftmanship, redux et redux toolkit sont très utilisés (si ce n'est, "la" solution de référence). Au dela de ça, c'est une "bonne techno" surtout parce que ça recrute bien sur du redux. Check les offres d'emploi, c'est pas toujours React Query ou Zustand ou autre qui sont demandés. Redux est encore très souvent demandé.
Et useReducer, ça sert à rien. C'est hyper couplé à React et c'est juste une pâle copie de Redux mais dans le monde de React en moins bien. Autant prendre le vrai Redux.
"Oui mais gneu gneu installer Redux , c'est lourd, tu installes une lib pour rien" ==> l'optimisation des packages, c'est pas l'enjeu ici. L'enjeu c'est de décrocher un job après une reconversion. Donc si faut installer redux (toolkit) pour montrer qu'on sait l'utiliser, j'encourage à le faire. C'est pas les opti sur les tailles de package qui vont payer les factures.
@@videv9858
Je dois confesser je connais pas du tout l'etat du marche du dev FE en france (puisque jy vis pas)
Apres, on va pas lancer un debat sur l'opti des tailles de package, c'est javascript, je vais devenir maboule en 2s tellement la gestion des dependences de ce langage me sort par tous les orifices ;)
Pour la partie git je suis pas tout à fait d'accord, perso quand je monte un petit projet avant la v1 je commit pas mal de "wip" etc et quand je sors la v1 je repars avec un repo clean depuis une branche orpheline, si le projet n'évolue pas, y a pas de commit.
je te partage des bonnes pratiques pour les projets en entreprise qui eux, évoluent toujours. Mais chacun a sa manière de faire. Là, tu vois la mienne :)
Quel est le nom de ce thème VScode orange et vert? (blanka?)
Cobalt2 🎩
@@videv9858 Super merci
Oui , frère vi perso j'aimerais revoir plus souvent ce genre de format et celui d'avant usage de usehook...et tous ce qui est concepte un peu flou sur reatct ou JavaScript ou même typescript
C'est pas un peu overkill de remplacer le button par Button? Ca prend la même place, mais on a un fichier en plus. Je vois pas trop l'intérêt dans ce cas précis.
L'intérêt c'est la réusabilité du composant. Et son plus gros avantage c'est que tu codes une fois et tu le réutilises plusieurs fois. Et si tu as besoin de changer le style de tous les buttons, t'as juste besoin de le changer à un seul endroit (dans Button.jsx) et non dans toouuuuuus les qu'il y a dans ton projet.
"Vous arrêtez ça hein ! " xD
Et encore là il est gentil .. tu l’as pas vu en mentoring 😂😂😂😂
@@dev1l0ux j’espère que t’as bien senti ma frustration quand j’ai dit ça 😭
Je regarde à nouveau ton live et je me dis que Philippe devrait trop prendre ton pack vidéo ..
C'est prevu. Pas pour cette fois malheureusement mais j'espere la prochaine session. Ceci dit, je serais curieux de connaitre la raison de ton commentaire, t'inquiete je ne suis pas susceptible, j'accepte toutes les critiques constructives. D'ailleurs depuis ce live, j'ai deja adopté de nouvelles pratiques comme par exemple l'utilisation systematique de git, et la refactorisation systematique en composants reutilisables. Par contre je n'ai pas encore switché sur Typescript, un gros morceau pour moi que je prefere laisser pour plus tard
@@jovialiste hey salut !
Pour te dire je trouve que tu as une bonne approche pour t’organiser.
Il te manque juste les best practices (que j’ai appris avec Vi à l’époque).
Dans sa formation on est sur 15 tickets et du ticket 7 au 15 on apprend beaucoup à organiser le code de façon claire et modulaire.
Pour moi si tu fais son projet, tu auras pas de mal à créer tes propres projets sur la base de sa structure.
Après j’avoue que son pack vidéo est à 700€ et c’est pas donné non plus donc ça se comprend .. mais moi lorsque commencé, je me suis dis qu’un plombier doit bien acheter du matériel avant de travailler.. donc ça fait un peu parti du matériel 😅😅
Sinon bravo ! Ton projet est bien codé, il te faut progresser en UI-UX mais à la limite c’est pas grave car en entreprise on a souvent les maquettes du designer donc tu as juste à connaître figma .. 😊
En tout cas continue ! Ne lâches pas le dev il manque plus grand chose 🏋️♂️
Ohhh my god ,j'ai rater le live ,j'ai pas reçu de notification,comment ca se fait
C'est ma faute, je l'ai mis en "non repertoriée" sans faire exprès 😭
C'est oas grave bro ,ca arrive
Vi ! Si je t’envoie mon repo et qu’il est clean, tu pourras m’embaucher pour de vrai ? 🥹🙃🥳
Si tu m'envoies ton repo et que je vois que c'est toi, même pas je l'ouvre 🫣
@@videv9858 🤣🤣🤣🤣🤣
@@videv9858😩😩😩