tmesser / DeckManager

BSD 3-Clause "New" or "Revised" License
5 stars 2 forks source link

Track revealed Cylons #39

Open MaxPeck opened 9 years ago

MaxPeck commented 9 years ago

They can't move to the same places we do. They can't draw as many skill cards as we can. They can't play as many skill cards into checks as we can.

But to implement any of that, we need to know who they are first. Vote Baltar for President.

tmesser commented 9 years ago

It is trivial to add a property to the back end to say 'this guy is a revealed Cylon', and this was actually in the program at one point. The issue is that it's really difficult to say how far this functionality should be integrated into DM. You could integrate Reveal functionality into the manage player form, but what should that mean, exactly? Are you going to restrict valid Locations to move to (which we currently don't)? Are you going to modify the valid initial draws? What does all that entail? Do we change the character name to something more Cylon-y?

I personally couldn't come up with a good answer for all of this so I ended up scrapping the idea completely, figuring that the moderator knows best and can handle it and the correct path forward was simply to provide him/her with the tools they needed to handle it the way they felt most appropriate. If we can agree upon some answers to these other questions we can talk about putting this in.

MaxPeck commented 9 years ago

I'd be okay with not restricting locations if we could at least sort Cylon locations to the top of that pulldown for revealed Cylons. As I mentioned in #14, they have a variable draw (technically, they have no skill set at all), but that's more a matter of reminding the user; similarly, it'd be nice if it displayed in the skill check form since it restricts cards drawn. Also, you need/want this for #30 since it appears in the generated image.

tmesser commented 9 years ago

OK, with relation to #30 we definitely need to add it to the manage player form on the front end and Player.cs on the back end, then. The way their variable draw works it's probably just better to eliminate their draws altogether and make the moderator draw it themselves - revealed toasters draw 2 cards of whatever they want, so that is 4*4 = 16 potential draws, which is well into 'who gives a shit' territory.

Regarding showing up in the Crisis form, what do you think would be the best way to express that? I'm kind of just a fan of renaming the character to "Model 2044" or something; this will make it obvious who's an overt toaster, divorce them from their previous card, and also be relatively easy for us to accomplish.

MaxPeck commented 9 years ago

I'm not a fan of renaming but it certainly works. I was just thinking of putting "CYLON" or "(max 1 card)" or something next to the Commit button (but not actually enforcing it).

tmesser commented 9 years ago

We'll work with "CYLON". Good suggestion.