search

ajhyndman / fire-emblem-working-title

A fledgling project wrangling Fire Emblem: Heroes stats
25 stars 6 forks source link

Update code to reflect new data format #147

Closed carsomyr closed 5 years ago

carsomyr commented 5 years ago

See title.

ajhyndman commented 5 years ago

Thanks so much for submitting this!

Just one comment: it looks like they changed the meaning as well as the name when translating "Restriction" fields to "CanUse"... which is why all the tests are failing.

I'll go ahead and merge, and take a stab at a fix if you don't get to it first.

The big question is probably whether to make a backwards incompatible change and change the name of the field in the stats data, or enumerate all the move/weapon types and compute the inverse set.