v0y / sport-tracker-with-acziwments

Sports tracker with achievements
http://sports-tracker-with-achievements.readthedocs.org
2 stars 0 forks source link

52 trasa z palca #93

Closed Cosiek closed 9 years ago

Cosiek commented 10 years ago

close #52 close #83

Cosiek commented 10 years ago

Jeszcze to trochę ogarnę.

Cosiek commented 10 years ago

Uff - jest. Oczywiście nie tykam się styli (bo u mnie działają tylko 3 strony z całego projektu - więc nie mam jak sprawdzić co popsłem).

W tym pr jest (oprócz kilku innych rzeczy nie związanych bezpośrednio z zadaniem), przebudowana wersja migracji dodającej besttime do usera. Nie było zadeklarowanej wartości domyślnej, więc wykopyrtywał mi się.

Jeszcze kilka rzeczy bym zmienił, ale to już na inne zadania. @v0y :elephant: :crocodile:

v0y commented 10 years ago

coś chyba nie działa do końca...

  1. klikam dodać ręcznie - pojawia się mapka, ale nic nie mogę z nią zrobić, nie mam jak wybrać trasy :[
  2. jak dodaję mapkę z pliku, pokazuje mi się czeska Praga - wcześniej chyba mapa z trasą otwierała się na trasie - nie popsułeś? ;)
Cosiek commented 10 years ago

U mnie działa bez zarzutów, z tym że ja korzystam z 52-dev który właśnie wypchnąłem.

Robię tak, bo nie byłem w stanie wymusić na projekcie, żeby działał z domyślnymi ustawieniami (zaczynam doceniać vagranta). Więc upraszczam go do tego co jest mi niezbędne do zrobienia zadania.

52-trasa_z_palca to 52-dev tylko wysprzątany. Zdziwił bym się gdyby moje porządki spowodowały błędy o krórych piszesz, ale proszę, sprawdź czy błędy powtarzają się na branchu dev. Ja nie mogę sprawdzić czy występują na głównym. :frowning:

To nie powinno być konieczne - ale warto spróbować skompilować coffee.

v0y commented 10 years ago

ok, zrobię tak. Poza tym mogę sprawić, aby projekt odpalał się na Vagrancie, jak nie będzie to zajebiście pracochłonne (nie powinno być). Zainteresowany?

Cosiek commented 10 years ago

Chyba przenoszenie projektu na vagranta z mojego powodu nie ma sensu. Jestem w stanie odpalić tą jedną stronkę która jest mi potrzebna, a znając życie dojście do porządku z vagrantem wcale proste nie będzie - ostatnio stawiałem dwa niby skonfigurowane - frajda to nie jest.

v0y commented 9 years ago

rebase z masterem

Cosiek commented 9 years ago

Rety - masz tu travisa?

rebase z masterem

Oznacza że mam zrobić? Czy że robisz?

v0y commented 9 years ago

jasne, że mam :) zrobiłem rebase z masterem przetestowałem, działa :+1:

na masterze jest readme, w którym opisałem odpalanie projektu i wywaliłem polecenie fab lets_rock. Teraz, jeśli przejdziesz instrukcję z readme powinieneś dać radę wszystko poinstalować. Z tego, co pamiętam, miałeś jakieś problemy z sudo.

Mam parę uwag, ale to są improvements a nie bugs. Kawał dobrej roboty!

v0y commented 9 years ago

możesz zaakceptować git accept-pr? ja nie mam tego polecenia

v0y commented 9 years ago

@Cosiek dobra, już ręcznie pofixupowałem komity z ^, Twoje dzieło jest na masterze

v0y commented 9 years ago

:cactus:

v0y commented 9 years ago

@Cosiek patrz, jak ładnie xD

Główny widok formularza:

screenshot from 2014-11-13 00 00 44


Dodawanie pliku:

screenshot from 2014-11-13 00 00 59


Dodawanie trasy z palca

(poprawiłem "Zapisz trasę" - wielka litera, ale nie chce mi się robić zrzutu ekranu) screenshot from 2014-11-13 00 06 10

Cosiek commented 9 years ago

:+1: