LibPlayerSpells-1.0

52.9k Downloads

Fix the upgrading path

Rainrider opened this issue ยท 4 comments

commented

We need to handle upgrading the tables we attach to the lib object in case of multiple different instances of LibPlayerSpells.

commented

Yes, I thought exactly about that. Currently there is no such a check in place.

commented

Do we really need to upgrade them ? I mean: if the previous version of LibPlayerSpells is too old (e.g. anterior to the version where the new format was introduced), just wipe the tables and start over with the new data.

commented

Which commit broke the API/data format and which was the library version before and after ?

Edit : there, 7 and 8.

So I think that clearing the data if the previous version was lower than 8 is what should be done.

commented

Closed with a2c474a