search

fabienjuif / subterra

Not finished clone of subterra board game
https://pedantic-lalande-d5e881.netlify.com/
0 stars 0 forks source link

myrtille action should be replayable #5

Closed fabienjuif closed 4 years ago

fabienjuif commented 4 years ago

Right now two @cards>pick can have 2 differents issues. This is not what we are looking for.

I think our engine should wrap functions like "pick", that:

  1. Do the random
  2. Call the action @cards>pick and give the random card picked

This way actions are replayable with only state.technical.actions

fabienjuif commented 4 years ago

I can take care of this today @WesleyRUCHAUD Fine to you?

WesleyRUCHAUD commented 4 years ago

Yes, it's fine for me. But I thinks that's not what we are looking for. We should have an action @cards>init that will generate your deckCards and the action @cards>pick should remove the first card from the deck

WesleyRUCHAUD commented 4 years ago

The last thing that is missing is to save the deckcard as to replay the game

WesleyRUCHAUD commented 4 years ago

the action @cards>pick remove the first card from the deck to set her as activeCard :)

fabienjuif commented 4 years ago

Hm I think you are right BUT we have to take care to not send all cards to players from server when we will be there!

WesleyRUCHAUD commented 4 years ago

so, we need to add deckCards in technical ?

fabienjuif commented 4 years ago

we will see later what we do for this, one step a time

fabienjuif commented 4 years ago

fixed in #6