fafranco82 / swdestinydb

Star Wars Destiny Deckbuilder
52 stars 26 forks source link

Adding 40 card limit to decks including CT4040 plot #345

Closed mlogie closed 3 years ago

mlogie commented 3 years ago

@fafranco82 this is step two to making that new plot I just submitted work. For me, I think there's 3 steps to making the 40/40 format work: Allow 40 points: achieved via the -10 plot Allow 40 cards: hopefully works in this fix, which checks if that plot is included, and if so let's you include 40 cards. Allow 2 plots if you have this plot. Currently don't know how to do that, but it's less urgent. Thank you as always, and apologies if I've broken something. I have literally never written anything in php before.

fafranco82 commented 3 years ago

Almost, but PHP is not the only validator. There is a Javascript validator, too, in app.deck.js file. Anyway, I will go on with your approach and finish that work ;)

I will comment here when it's done.

fafranco82 commented 3 years ago

Changed