Closed Zac-Z88 closed 9 years ago
aroquemaurel
commented
9 years ago J'ai pas encore regardé, vais le faire, mais juste, tu sais faire un git rebase ?
Il faudrait ajouter le #noIssue dans chacun de tes commits :p
Si tu ne sais pas faire les rebase, je m'en occupe pas de soucis !
aroquemaurel
commented
9 years ago D'un point de vue fonctionnel, c'est nickel ! GG à toi !
Le problème venait d'où du coup ?
Zac-Z88
commented
9 years ago Le problème venait du fait que spring security n'enregistrait pas le login qui ce fait au niveau du userController j'ai du déclencher la procédure d'identification avec spring security avant de d'effectuer le login.
aroquemaurel
commented
9 years ago Okey :)
Du coup, tu veux que je fasse le rebase ?
Zac-Z88
commented
9 years ago Je vais essayer de le faire moi même, si je n'y arrive pas je te ferais la passe :P
aroquemaurel
commented
9 years ago https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History section Changing Multiple Commit Messages
En gros, faut que tu fasse un git rebase -i (rebase interractif), tu notifie chacun des commits en « edit », et tu changes message par message, c'est-à-dire tu modifie ton message, puis git rebase --continue
Une fois que c'est bon, tu vérifie que ton historique te va bien (git log), si c'est le cas, tu fais un git push -f origin sprint02/46-AmendLoginFeature. C'est ce push ou il faut faire attention, ça peut être dangereux… ;)
aroquemaurel
commented
9 years ago Si tu galères, aucun soucis pour que je m'en occupe, ça me prendra 5mn :)
aroquemaurel
commented
9 years ago Good game ;-) Si le build est vert, tu peux faire le merge !
Le bug du login a été corrigé + en bonus la confirmation d'inscription par mail :P J'ai aussi créé le service d'initialisation donc si vous avez des trucs à ajouter c'est là qu'il faut le faire dorénavant je vous demanderais par contre de jeter un coup d'oeil aux test d'intégration, j'ai mis des commentaires dessus si quelqu'un peut m'éclairer ;)