corentindesfougeres / gatcha-prototype Goto Github PK

If the waiting list is empty, the combat manager doesn't specifically ask for actions, it just fail. Sould be corrected by adding a verification in Act() that if waiting list is empty it ask another component on the spirit that will handle the choice of action.
That other component should be divided in two kind : npc which ask to an IA and pc which ask to the player. The chosen action is put in WaitingList[0] then Act() proceed normally.

IA turn doesn't seem to removes it from the acting list, so the CM loop is stuck there

No idea if the system is working, should be tested

Events don't work as they are set up right now, check how to set them up and correct it. Notably for the coupling UI / logic

No IA right now, so the combat block when it's an opponent turn.

Separate those two states and give them their each their own if statement and logic in the Update()

MoveBehaviour would be easier to use if it wasn't a Monobehaviour like planned initially.

the actions lists in Moves and Alterations are not displaying correctly, and thus impossible to fill.

Not sure the inheritence in the action/actor system is setup correctly. Sould be virtual classes / methodes ?

OdinSerialised properties lose their values on play.

Make a general StrorageManager that keeps in memory UnitsBehaviour of all owned spirits, but for that might need to switch UB as a non monobehaviour.

The CombatManager update loop should be functionning, but the script lacks methods to initialize and so it can't be tested. That should be added asap.

Can't test the combat system without a way to input the desired actions. Should include both the UI to interact with and the methods to enter the choices in the waiting list. Should also pause the combat loop if there is nothing in the waiting list until something is chosen.

CUIM would be a perfect fit for a stare machine, and a good way to practice them, so let's turn it into one.