0maczel / pz

Projekt realizowany w ramach zajęć "Programowanie Zespołowe" na WFiIS w semestrze letnim 2015/2016.
0 stars 0 forks source link

Implementacja serwisów niezbędnych dla zapisywania pomiarów sensorów. #20

Closed dawidkomorowski closed 8 years ago

dawidkomorowski commented 8 years ago

Podstawowy przypadek użycia REST API monitora to proces zapisywania pomiarów nadawanych przez sensory.

Należy zaimplementować klasy serwisowe i ich metody pozwalające na:

Dodatkowo należy odpowiednio zaktualizować dokumentację REST API.

Link: #16

Izzy4me commented 8 years ago

Szybkie pytanie: Czy w tym momencie jakaś dodatkowa konfiguracja/instalacja pakietów jest wymagana, czy wystarczy jak odpalę projekt w Eclipsie, Maven wszystko dociągnie i projekt będzie nadawał się do odpalenia?

dawidkomorowski commented 8 years ago

Maven powinien wszystko ogarnąć.

dawidkomorowski commented 8 years ago

@Izzy4me Hej, mam takie pytanie, bo przypisałeś się do tego issue, a nie wiem czy w sumie coś w tym obszarze już zrobiłeś? Jeśli nie to ja mogę się za to dziś zabrać, gdyż jestem w temacie kończąc właśnie #21.

Izzy4me commented 8 years ago

Hej, Pushne branch ze swoimi kodem ok. 20. Jeśli uznasz, że coś jest nie tak to zrób z nim co chcesz :P

Pozdrawiam, Mateusz 10 maj 2016 17:42 "dawidkomorowski" notifications@github.com napisał(a):

@Izzy4me https://github.com/Izzy4me Hej, mam takie pytanie, bo przypisałeś się do tego issue, a nie wiem czy w sumie coś w tym obszarze już zrobiłeś? Jeśli nie to ja mogę się za to dziś zabrać, gdyż jestem w temacie kończąc właśnie #21 https://github.com/0maczel/pz/issues/21.

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/0maczel/pz/issues/20#issuecomment-218198399

Izzy4me commented 8 years ago

Btw: chcemy mieć jakieś konkretne testy pokrywające REST API (mocki czy jak to się zwie w Javie?). Zawsze (także teraz) ograniczałem się do testów przez Postman'a, ale warto spytać :D

dawidkomorowski commented 8 years ago

Tak myślałem, że warto by napisać jakieś testy do serwisów też.

dawidkomorowski commented 8 years ago

@Izzy4me jak wygląda status tego taska? Bo chyba w czwartek mamy pokazać prototyp systemu a tu jeszcze nikt chyba tych komponentów nawet nie próbował skomunikować ze sobą.

Izzy4me commented 8 years ago

Działająca funkcja obsługująca tego POST'a (sprawdzana postmanem) znajduje się w osobnym branchu, którego pusnąłem do zerknięcia w zeszłym tygodniu. W weekend widziałem jednak, że (invader92 dodał komentarz) chcecie parsować jsony na innym poziomie, więc jeśli ta wersja jest nieodpowiednia, postaram się zmienić ją dziś wieczorem.

2016-05-17 10:33 GMT+02:00 dawidkomorowski notifications@github.com:

@Izzy4me https://github.com/Izzy4me jak wygląda status tego taska? Bo chyba w czwartek mamy pokazać prototyp systemu a tu jeszcze nikt chyba tych komponentów nawet nie próbował skomunikować ze sobą.

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/0maczel/pz/issues/20#issuecomment-219653808

dawidkomorowski commented 8 years ago

@Izzy4me Pojawił się label 'question', co się za tym kryje?

Izzy4me commented 8 years ago

Zerknąłem na komentarz @invader92 i czy powinno iść to bardziej w tym kierunku? http://pastebin.com/eYm5CpM1

Niestety obecnie nie widzę, żeby w tej postaci dto było przyjmowane przez funkcję (albo źle wysyłaniem raw json'a w postmanie).

dawidkomorowski commented 8 years ago

To co dałeś na pastebina jest jak najbardziej OK. Jedynie niepotrzebnie ustawiane są daty creation i update - konfiguracja encji jest taka, że hibernate sam je ustawia podczas operacji na bazie.

Co to tej historii z dto i postmanem to nie wiem czy zrozumiałem. Ja te enpotiny co pisałem testowałem zwyczajnie z przeglądarki - napisałem 5 linijek JavaScript, które robią JSON a potem wykonują XHR i działało jak trza ;)