search

Vunovati / uskoci

4 stars 2 forks source link

Eventi komunikacija + još par evenata #39

Closed Vatavuk closed 12 years ago

Vatavuk commented 12 years ago

Napravljena dvosmjerna komunikacija između back - front za evente.

Rješeni eventi: Will of God, Storm, Spy i SpyGlass.

Moram dodati testove za Spy i SpyGlass koji ukljucuju i komunikaciju.

Naknadno cu malo ispolirati kod. Za sad su svi eventi i komunikacija stavljeni u jednoj metodi "resolveEvent", budem to rascijepkao da bude što intuitivnije i preglednije za eventualno kasnije dodavanje/mijenjanje evenata.

buildhive commented 12 years ago

Vladimir Adamić » uskoci #77 FAILURE Looks like there's a problem with this pull request (what's this?)

buildhive commented 12 years ago

Vladimir Adamić » uskoci #78 SUCCESS This pull request looks good (what's this?)

Vunovati commented 12 years ago

Dodao sam samo par komentara da malo prodiskutiramo kod prije merganja. To mi se čini kao dobra praksa za inače ...

buildhive commented 12 years ago

Vladimir Adamić » uskoci #80 FAILURE Looks like there's a problem with this pull request (what's this?)

Vatavuk commented 12 years ago

Neznam zakaj nije prošo, meni u mavenu prolazi sve testove, aj pls Vlado pogledaj što je....

buildhive commented 12 years ago

Vladimir Adamić » uskoci #82 SUCCESS This pull request looks good (what's this?)

buildhive commented 12 years ago

Vladimir Adamić » uskoci #86 SUCCESS This pull request looks good (what's this?)

Vunovati commented 12 years ago

Sad pazi kod fetchanja ove grane s Githuba

Znači prvo (iz tvoje rulesImpl grane)

git fetch origin git pull origin rulesImpl

Vunovati commented 12 years ago

Radi daljnje korake u novom branchu. Za svaki event napravi novi branch iu novi pull request ...