Closed Maxopoly closed 8 years ago
Can one of the admins verify this patch? Type 'ok to test' to test.
No comment means I can merge and do whatever I want?
Probably :P
@rourke750 Anything?
@Maxopoly How'd testing go?
Ok, this is completly tested and both flatfile and sql are working with all of this. While I was on it I also fixed a ton of issues with flatfile storage, it was barely functional.
Unless there are any objections, I'd say we merge this.
If you've tested it I see no issues here, code looks clean, good refactors.
Rework flatfile saving to use yml, add killer and death time to pearls and storage, improve usage of interfaces to no longer duplicate code
Untested so far, but will test before merging