Advanced-Slime-World-Manager icon indicating copy to clipboard operation
Advanced-Slime-World-Manager copied to clipboard

API imrovements

Open Gerolmed opened this issue 5 years ago • 9 comments

Thinks I would consider to add are:

1. Improve slimeproperties

Make slimeproperty type more generic. Allow developers to register custom property resolvers, to allow attaching custom/customly structured data to worlds.

2. Add async methods

Add async methods for world loading and exist checks. Using callbacks as a return to make it more userfriendly for beginners

3. Make API more userfriendly

Recreate the old property builder, but allow using generic properties for building. [More ideas to add here as a comment pls]

4. Better access to loaders

Currently to create a SlimeLoader a slime loader class has to be rewritten from scratch. Either make the default loaders available, by moving them to the API project. Or maybe add some utils class to create them in the api and register the loaders from the plugin module in there. Several projects/plugins would do better by using there own seperated storage, but manytimes this is not the case because people dont want to bother to search for current implementationso or are scared off by having to implement a loader themselves not knowing where to find the current implementations.

Gerolmed avatar Jul 29 '20 09:07 Gerolmed

@Paul19988 should I create a new branch for this

Gerolmed avatar Jul 30 '20 17:07 Gerolmed

Yeah if you create a separate branch we can PR it afterwards

Paul19988 avatar Jul 30 '20 17:07 Paul19988

ok great

Gerolmed avatar Jul 30 '20 17:07 Gerolmed

It seems that nine months have passed, do you still plan to solve this problem?

BingYanchi avatar Apr 03 '21 14:04 BingYanchi

It seems that nine months have passed, do you still plan to solve this problem?

This is not an actual issue but rather API improvements that specifically Gerolmed is working towards, it's not intended to be "solved" as it's not a problem

Paul19988 avatar Apr 03 '21 14:04 Paul19988

It seems that nine months have passed, do you still plan to solve this problem?

This is not an actual issue but rather API improvements that specifically Gerolmed is working towards, it's not intended to be "solved" as it's not a problem

I am using SSB-SlimeWorldManager (https://github.com/BG-Software-LLC/SSB-SlimeWorldManager/issues/2) encountered an error.

I don't know much about plugins, but the author of SSB-SlimeWorldManager said it was due to ASWM problems. Maybe you can give me an answer.

BingYanchi avatar Apr 03 '21 14:04 BingYanchi

What exact feature are you referring to? I think slime properties have already been changed.

Gerolmed avatar Apr 03 '21 14:04 Gerolmed

What exact feature are you referring to? I think slime properties have already been changed.

It seems that there was an error in creating and deleting the world, but I don't know the reason for it.

BingYanchi avatar Apr 03 '21 15:04 BingYanchi

Hello any updates about the ETA of this? I am really interested in async/multithreading methods :D

Blaxor avatar May 07 '21 08:05 Blaxor