search

EldenRingDatabase / erdb

JSON schema and data parser for ELDEN RING
MIT License
47 stars 8 forks source link

Armor missing #2

Closed CyberGiant7 closed 2 years ago

CyberGiant7 commented 2 years ago

Hey, thank you for you work. I'm using the json database and I just noticed that the Godskin Noble armor has been swapped with that of the Godskin Apostle, while the Godskin Apostle armor is missing

Phil25 commented 2 years ago

Hey good catch, thanks.

I took a break from erdb but when I get back to it around next week I'll check it out. Speaking of which, I'm glad that you're using it but do mind that it's still WIP and things will change.

Thanks for the report.

Phil25 commented 2 years ago

Fixed in https://github.com/EldenRingDatabase/erdb/commit/9cc9a4601a1e2ccc528c06e7aa1a04d030c5267e

Also corrected the data for 1.07 only. Previous versions will have the wrong armor for the time being until generators are more complete and I'll downpatch the game to update them.

Phil25 commented 1 year ago

All versions are now updated.