Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestions for a better DazzleConf #33

Open
BlueTree242 opened this issue Feb 8, 2023 · 1 comment
Open

Suggestions for a better DazzleConf #33

BlueTree242 opened this issue Feb 8, 2023 · 1 comment
Milestone

Comments

@BlueTree242
Copy link

BlueTree242 commented Feb 8, 2023

DazzleConf is a very useful library and makes config cooler and easier, but now let's make it even better.

First of all, change how the interface is processed, instead of current way it works it could be where it loads the interface, and looks for method with default which would have a special default, then it would store these extra defaults when generating the config. and also ability for variable methods, which is not related to the config but to make working with dazzleconf's way of configuration easier. It is simple, we can pass variables with identifier, and there would be an annotation like VariableMethod and requires the identifier, the method then is no longer a configuration option and is just a variable used by default methods to know what they should return exactly. and third is Double order because integers between 2 integers are always finite, but decimals are always infinite. And lastly. it is annoying when you have to use DefaultObject so replace/or add a feature where you can add your own default function like when adding config serializable, identified by a string. And most important of all of this, fixing the bug with hashmap. Also, a converter which allows previous options to change location, for example if the type was previously a list.. but now a map, and the map contains the list, it would move the list into its new location. And also make sure all of this doesn't break if their parents are lists.

This will make dazzleconf easier & more beginner friendly. it will reduce the code you have to write, and make it look better, and will also make the library more advanced. So i hope you add this in the next releases, maybe 2.0.

@A248 A248 added this to the 2.0.0 milestone Jul 15, 2023
@A248
Copy link
Owner

A248 commented Jul 15, 2023

All very good suggestions, thank you. I finally understood what you meant with variable methods, since I recently encountered this exact issue while wanting to improve LibertyBans' incorporation of addon configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants