search

Maciekek / ProjektZespolowy2015

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

Add part containing registration form and pop-up which will be shown when user try to use existing login. #32

Closed mbednarczyk closed 9 years ago

mbednarczyk commented 9 years ago

Ok dodałem część odpowiedzialną za tworzenie konta(czyli najprostszy formularz), do tego jest też cześć odpowiedzialna za wyświetlenie informacji w postaci pop-upu, gdy użytkownik będzie próbował się zarejestrować z już istniejącym loginem.

Inputy posiadają odpowiednie tagi tj.: data-ng-model="login" i data-ng-model="pass"

Nie wiem tylko czy przycisk "rejestruj" nie powinien miec takiego tagu: button z tagiem data-ng-click="register()" ale dodanie tego zajmie sekunde w razie co.

Maciekek commented 9 years ago

Powinien mieć:)

Sent from phone 28 paź 2014 12:58 "Marek" notifications@github.com napisał(a):

Ok dodałem część odpowiedzialną za tworzenie konta(czyli najprostszy formularz), do tego jest też cześć odpowiedzialna za wyświetlenie informacji w postaci pop-upu, gdy użytkownik będzie próbował się zarejestrować z już istniejącym loginem.

Inputy posiadają odpowiednie tagi tj.: data-ng-model="login" i data-ng-model="pass"

Nie wiem tylko czy przycisk "rejestruj" nie powinien miec takiego tagu: button z tagiem data-ng-click="register()"

ale dodanie tego zajmie sekunde w razie co.

You can merge this Pull Request by running

git pull https://github.com/Maciekek/ProjektZespolowy2015 dev/#22

Or view, comment on, or merge it at:

https://github.com/Maciekek/ProjektZespolowy2015/pull/32 Commit Summary

  • Add part containing registration form and pop-up which will be shown when user try to use existing login.

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/Maciekek/ProjektZespolowy2015/pull/32.

mbednarczyk commented 9 years ago

ok to dodaje

Maciekek commented 9 years ago

A spróbujesz napisać do tego test?:) Dokładnie w tym teście: WebApp\test\e2e\scenarios.js

Dodaj test, który sprawdza czy po kliknięciu przycisku "zarejstruj się już teraz" przeniesie nas na strone na której będzie element z napisem: "Rejestracja zajmię tylko kilka sekund!"

Być może trzeba będzie poczekać na ten element w teście;)

Jak masz ochotę to pokombinuj:) Tutaj masz mały tutorial: http://angular.github.io/protractor/#/tutorial Wydaje mi się, że ta wiedza, która tu jest wystarczy, żeby to zrobić:) Jak coś to pisz

mbednarczyk commented 9 years ago

Ok popraiwe to data-*, test tez spróbuję napisać :). Ale to zajmę się najprawdopodobniej późnym wieczorem.

mbednarczyk commented 9 years ago

Jak bede robił test to robic nowy branch czy zostac na tym ?

Jankkol commented 9 years ago

branch == task, więc jak w ramach tego taska to na tym ;)

W dniu 28 października 2014 17:39 użytkownik Marek <notifications@github.com

napisał:

Jak bede robił test to robic nowy branch czy zostac na tym ?

— Reply to this email directly or view it on GitHub https://github.com/Maciekek/ProjektZespolowy2015/pull/32#issuecomment-60787731 .

Maciekek commented 9 years ago

w tym tasku czekamy na testy tak?:) bo duży PR już jest do wmergowania;)

Maciekek commented 9 years ago

zamykam PR do czasu testow

mbednarczyk commented 9 years ago

Test dodany i działający :). Otwiera PR.

Maciekek commented 9 years ago

git