search

AfricaBank / front_end_officiel

0 stars 0 forks source link

Sidebar #12

Closed Berthonge21 closed 4 months ago

Berthonge21 commented 4 months ago

Check the code if everything looks good or not I've added the hover and fixed the Link directions of every items If there is not modification concerning the sidebar merge it on the main branch

Berthonge21 commented 4 months ago

Fait attention sur la gestion des conflicts

Abdi714 commented 4 months ago

Bonjour ! bien reçu.

Berthonge21 commented 4 months ago

Je te l'avais signifié de faire attention lors du merge parce que j'avais vue des conflits Et ça n'a rien avoir avec un pull parce que tu ne peux pas faire un pull dans une branche que tu ne travaille pas je peux pas faire un pull sur le main alors que je ne travaille pas sur le main y'avait juste une incoherence sur le code du main Et j'ai pas voulu fixer cela pour que tu voies et t' apprennes en même temps @Abdi714

Abdi714 commented 4 months ago

Et bah justement, c'est ce qu'il fallait faire. Moi j'ai bein étudié les incohérences et j'en ai corrigé le maximum. Mais ce qu'i faut noter c'est ton code qui devait s'adapter au code base et non le contraire. Tu n'as pas besoins de trvaillé sur le main pour effectuer un git pull. la raison pour laquelle tu dois le faire, c'est juste pour recuperé le code base de cette branche qui est la branche principale avec une structure de code déjà approuvé par tout le monde. Donc le git pull via cette branche a pour objet de recopier les derniers misees à jour du main sur la branche avec laquelle tu travailles. En procedant de cette manière, cela va éviter les conflits dans un premier temps, et secondo, tu rajoutes ton code en poursuivant l'existant qui est le main. Si besoins, on peut faire un call pour clarifier tout ça car j'ai comme l'impression que nous n'avons pas la même compréhension de certains concepts.

Abdi714 commented 4 months ago

je vais merger ton code d'une autre manière car, bien que j'ai fixé les conflits, j'ai eu d'autre complications à runner le projet aprés le merge. J'ai du faire un revert pour retourner à la version ancienne du main. Je vais juste recupérer le code du sidebar depuis ta branche Sidebar et l'integréer en local puis effectuer un un nouveau commit puis un push. De cette façon, nous aurons ton code qui va s'alligner avec le code de la branche main actuelle partagé que dispose touts les autres developpeurs.

Berthonge21 commented 4 months ago

Je t'appelle là ça passe pas