Conversation
bnszky
left a comment
There was a problem hiding this comment.
Powiem tak: Świetnie napisana logika, podoba mi się, że zabezpieczyłeś, gdy w trakcie przejścia ściezki ventu zablokujemy przejscie, to kapibara nie przejdzie. Na ten moment jest to świetna podstawa. Właściwie nie mam do czego się przyczepić. Jedyne o co prosze przed mergem, to żebyś dodał te zespawnowane venty i kapibare do przykladowego pliku json (np. level 1), zaktualizowal room.schema.json i dodał wczytywanie kapibary i ventów bezpośrednio z pliku a nie hardcode values. Animacje będą dodawane później, więc tym się nie przejmuj.
| this.ventState.spawnInitialVents(); | ||
| this.spawnCapybara(); | ||
|
|
There was a problem hiding this comment.
tutaj wczytuj z pliku json pozycje "zaproponuj jak bedzie przechowywany vent i capybara w json"
|
Instrukcja rebase
sprawdzamy czy wszystko poprawione gdy wszystko ok, to powtarzamy proces az procedura poprawy commitow sie skonczy i wracamy znowu do punktu 3 |
bnszky
left a comment
There was a problem hiding this comment.
należy zrobić rebase i pobrać najważniejsze zmiany z maina
8aff018 to
4dfd7df
Compare
Nie ręczę za poprawność pozycji kapibary i wentyli, bo jeszcze walczę ze zmianami na froncie. Reszta powinna być git.