Closed Maciekek closed 9 years ago
Póki co jedyne do poprawy to przekierowanie po podaniu błędnego hasła. Aktualnie kieruje pod /main ale taka część nie istnieje.
Widze, ze commit jest grubyy :-) sprawdze jutro co i jak, nie chce robic tego na szybko. 8 lis 2014 20:02 "Marek" notifications@github.com napisał(a):
Póki co jedyne do poprawy to przekierowanie po podaniu błędnego hasła. Aktualnie kieruje pod /main ale taka część nie istnieje.
— Reply to this email directly or view it on GitHub https://github.com/Maciekek/ProjektZespolowy2015/pull/40#issuecomment-62270996 .
@mbednarczyk już poprawione i potwierdzone testem
Ok, zostawiłem komentarze po CR. Zobacz co o nich myślisz.
Takie małe info, commit zawiera również działające rozwiązanie tego taska https://github.com/Maciekek/ProjektZespolowy2015/issues/26 :)
Wprowadziłem poprawki oprócz testów. Wymagają one przemyślenia, jak to chcemy testować, Zajmę się tym, ale chciałbym, żeby to już było wmergowane, żeby kogoś przypadkiem nie trzymać
Na testy wystawie osobnego taska (chodzi mi o to ze duzo zeczy sie powtarza, bo sa to testy e2e, czyli trzeba przejsc caly proces "klikania") Można coś tam jeszcze pozmieniać ale to trzeba mieć chwilę na to, której teraz brak. Ważne, że działają i testują. Lepiej jak testują za dużo niż za mało
Ktoś zmerguje, albo da znac co jest nie tak?
hallooo
Nie dam rady teraz sprawdzić, jeżeli ktoś inny może to proszę o to. Jeżeli nie to postaram się jutro.
2014-11-18 22:28 GMT+01:00 Maciej Kucharski notifications@github.com:
hallooo
— Reply to this email directly or view it on GitHub https://github.com/Maciekek/ProjektZespolowy2015/pull/40#issuecomment-63548270 .
25
Dosyć gruby commit, dużo pracy Przejrzyjcie to uważnie bo wiele się tam dzieje Jak macie jakieś pytania to piszcie w komentarzach do kodu;)