pixelsoccupied / 345_California

Popular board game "PANDEMIC" done in C++.
0 stars 1 forks source link

Role Card Functionality #102

Open darkheros12 opened 7 years ago

darkheros12 commented 7 years ago

I dont know if all the functionality of Role Card are been implemented in our project. Can you clarify me on the matter? @pixelsoccupied

pixelsoccupied commented 7 years ago

Yap. Gonna use the new PlayerRoleCard class. Need some GUI help though.(eg. one role allows for 8 cards) Gonna add issues when I go thru all.

BTW take a look into the new files I just uploaded. Separated a action to the playerAction. Just the directFlight for now. The rest are using the old way so that you can compare it. Take a look into buttons.cpp, setPlayerActionsAndRole() in board.cpp and the new attributes in Player (commented). @darkheros12 @darkheros12

darkheros12 commented 7 years ago

make sure you PULL REQUEST to master, and if there is anything to fix, I will try to fix it or you

pixelsoccupied commented 7 years ago

i can't pull request cause my GUI files are different than yours @darkheros12...might mess up the whole thing...I can add only the specific non GUI files though...that's cool?

darkheros12 commented 7 years ago

Its fine

darkheros12 commented 7 years ago

Can you precise with file you change or modify please.

eg:

XYZ.cpp and XYZ.h etc...

pixelsoccupied commented 7 years ago

https://github.com/pixelsoccupied/345_California/tree/pixelsoccupied-dataModelChanges

All the changes ^ @darkheros12

MatthewChrobak commented 7 years ago

@darkheros12 @pixelsoccupied is this resolved? Can we close this?

pixelsoccupied commented 7 years ago

@MatthewChrobak check this out! https://github.com/pixelsoccupied/345_California/issues/130