search

openpkw / openpkw-weryfikator

Backend dla aplikacji weryfikującej wyniki wyborów
2 stars 8 forks source link

Qr service #13

Closed pogos closed 8 years ago

pogos commented 8 years ago

Dodałem serwis REST do odbierania QR, parsowania go oraz zapisu danych.

Pull request zawiera również zmiany z branch'a OW-R-2.

mrozim78 commented 8 years ago

Na szybko uwagi:

  1. Zbudował bym warstwie service w controller zrobil by wywolanie tylko service.
  2. Zastanowil bym sie jakie przyjac miejsce dla modelu przesylanych jsonem
  3. To samo w co poprzednim request ustalil jak bedziemy budowac linki i jak przesylac parametry w rest.
  4. Jak zbudujemy warstwe service bedzie mozna podejsc tu do unit testow dla tego kodu.
SebastianCelejewski commented 8 years ago

Zmergowałem zmiany Remka z masteram, a teraz czekają na zmergowanie zmiany Łukasza z masterem. Więc ten branch będzie mocno nieaktualny i trzebaby do niego wgrać aktualnego mastera.