Maciekek / ProjektZespolowy2015

MoneyGiver
http://178.62.111.179/
0 stars 2 forks source link

Implementacja tworzenie konta użytkownika #23

Closed Maciekek closed 9 years ago

Maciekek commented 10 years ago

Połączenie kontrolera (który trzeba utworzyć) z widokiem na stronie do tworzenia nowego konta użytkownika.

Kontroler ma zbierać dane ze strony (login i hasło) I wysyłać je do serwera. Serwer powinien to zapisać.

Przyda się również walidacja tego. Można to zrobić tak, że po wpisaniu swojej propozycji loginu i przejściu do wpisywania hasła aplikacja wysyła do serwera ajaxem dane loginu i serwer zwraca czy ten login jest wolny czy zajęty. Od tej odpowiedzi zależeć będzie zachowanie widoku.

Jankkol commented 10 years ago

Wziąłbym chętnie tego task'a, ale nie wiem do końca jak byśmy chcieli tworzyć serwisy i obiekty. Ja bym widział takiego user'a jako obiekt z polami (login, hasło, rola) i metodami które można by na nim wywołać (póki co podstawowy CRUD). Kontroler zostawił bym tylko jako komunikator między serwerem a klientem i tak naprawdę przekazywałbym z niego od razu dane do serwera - który zrobi całą resztę. Jak wy to widzicie? @Maciekek ? Może chciałbyś zrobić to jako pierwszy? Nie wiem czy kiedyś robiłeś, czy masz już może jakiś pomysł na to (tylko mało czasu)?

Maciekek commented 10 years ago

Jak chcesz to możesz śmiało robić. Co do komunikacji to tak to trzeba zrobić. Co do "obiektu", myślisz tutaj po javowemu. Nie przezuj w obiekcie metod, bo to nie ma sensu. Bo i tak i tak wyślesz te dane na odpowiedni routing postem. Nie nazywałbym tego obiektem. Po prostu json z odpowiednimi danymi. Kontroler musi mieć też odpowienią funkcjonalnosć reakcji na odpiedzi serwera.

Rób jak uważasz za słuszne, a jak zrobisz to zobaczmy jak to wyszło.

Jankkol commented 10 years ago

Ok, zacznę w wolnej chwili i zobaczymy co mi z tego wyjdzie, ale jeżeli będzie to blokowało innych to też śmiało bierzcie.

W dniu 30 października 2014 14:35 użytkownik Maciej Kucharski < notifications@github.com> napisał:

Jak chcesz to możesz śmiało robić. Co do komunikacji to tak to trzeba zrobić. Co do "obiektu", myślisz tutaj po javowemu. Nie przezuj w obiekcie metod, bo to nie ma sensu. Bo i tak i tak wyślesz te dane na odpowiedni routing postem. Nie nazywałbym tego obiektem. Po prostu json z odpowiednimi danymi. Kontroler musi mieć też odpowienią funkcjonalnosć reakcji na odpiedzi serwera.

Rób jak uważasz za słuszne, a jak zrobisz to zobaczmy jak to wyszło.

— Reply to this email directly or view it on GitHub https://github.com/Maciekek/ProjektZespolowy2015/issues/23#issuecomment-61091786 .

Maciekek commented 10 years ago

To jak zaczneisz to daj znać, zeby było info ze ktos juz to zaczal

Jankkol commented 10 years ago

Zaczynam taska.

Jankkol commented 10 years ago

Mam problem z wstrzykiwaniem serwisu do kontrolera. Może ktoś będzie wiedział co jest nie tak. Wrzuciłem kod na branch dev/#23

Maciekek commented 9 years ago

Rozumiem co chciałeś zrobić, ale angularJS działa trochę inaczej. Rozumiem, że taka zmiana myslenia moze być na początku trochę trudna dlatego zrealizowałem tego taska po swojemu. Mam problem z testami unitowymi, musze tutaj zasięgnąć języka u kogoś bardziej doświadczonego. Za to testy e2e działają pięknie.

Zobaczcie jak to zrobiłem, jak będziecie mieli pytania i uwagi to mówcie. Ja też sie angulara uczę, więc może coś można lepiej zrobić?

A, kod pisałem jak było już po 12 i po święcie, także mogą się tam pojawić jakieś dziwne rzeczy. Jak coś to poprawie to;)

Jankkol commented 9 years ago

Dokladnie :-) zobacze to popoludniu i dam feedback. 2 lis 2014 12:26 "Maciej Kucharski" notifications@github.com napisał(a):

Rozumiem co chciałeś zrobić, ale angularJS działa trochę inaczej. Rozumiem, że taka zmiana myslenia moze być na początku trochę trudna dlatego zrealizowałem tego taska po swojemu. Mam problem z testami unitowymi, musze tutaj zasięgnąć języka u kogoś bardziej doświadczonego. Za to testy e2e działają pięknie.

Zobaczcie jak to zrobiłem, jak będziecie mieli pytania i uwagi to mówcie. Ja też sie angulara uczę, więc może coś można lepiej zrobić?

— Reply to this email directly or view it on GitHub https://github.com/Maciekek/ProjektZespolowy2015/issues/23#issuecomment-61402929 .