Advanced search - Githubissues

airlog / flavority-api

RESTful API for the tastiest website

0 stars 1 forks source link

Advanced search #32

Closed V-I-S closed 10 years ago

V-I-S commented 10 years ago

Hej, tu jest wstępna wersja przeszukiwania po produktach. Sugeruję jeszcze tego nie mergować.

Mam pewne wątpliwości co do niektórych operacji czy tak gładko przejdą, ale bez webapp nie bardzo miałem pomysł jak to debugować. Jeśli tam jest już zacommitowany odpowiedni GET wywołujący nowy advancedSearch w API - dajcie znać.

PS. Co jest do zrobienia jeszcze? Bo noc jeszcze młoda, mogę się za coś wziąć ;)

V-I-S commented 10 years ago

Shiiit...No ja pierdziele, commentsy przypadkiem wrzuciłem, jeszcze kuźwa z nierozwiązanym konfliktem, sorry, zaraz coś z tym zrobię.

V-I-S commented 10 years ago

Aaa...I jeszcze te komentarze są w dwóch językach - te co chciałem by zostały - po angielsku, tymczasowe po polsku.

:mag: Co zrobić, aby szybko zdiscardować ten syf co narobiłem w comments.py na tym branchu? Mogę przekopiować cały kod z originowego mastera, ale domyślam się, że to nie najpiękniejsze rozwiązanie...?

robin92 commented 10 years ago

Dobra, jest problem:

skomitowałeś merge'e >>>>>> HEAD, czy odpalałeś chociaż raz kod, który napisałeś? z tym działać nie będzie
praca z gitem nie polega na wciskaniu wszytkiego do jednego brancha; po tym co napisałeś ostatnio powinieneś (tak jak mówiłem) przejść na origin/master potem utworzyć nowego brancha i pisać na nim
nie dogadałeś się z Asią co otrzymujesz w zapytaniu

Ogólnie ten PR zamykam bo nic z tym nie zrobimy. Żeby kod nie przepadł musisz pojawić się trochę wcześniej dzisiaj to pomogę Ci ogarnąć ten PR żeby kod nie przepadł.