-
Notifications
You must be signed in to change notification settings - Fork 160
Add support for Property option for dropdown in oneconfig
#1132
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
Conversation
|
I mean it works, and it is somehow even worse code than the rest of this compat layer, so why not |
I am pretty aware it is terrible code but it's going to be rewritten with OneConfig V1 anyways and it's a pretty critical error (the compat layer literally does not function) For future reference: does |
|
MoulConfig supports |
ah. well oneconfig is a bit brittle code-wise and it would be a bit of a pain to support this (i think i would have to make a property wrapper for every config option and i don't want to do that), so ill wait till V1 for that. For now, though, this PR should be merged as the compat layer just doesn't function rn |
Pretty sure i got told like at least a year ago that oneconfig would make those kinds of things easier soon, so i would rather still get it right in the current system. I will rewrite the entire thing myself anyway to be not neu exclusive. |
Property option for dropdownProperty option for dropdown in oneconfig

No description provided.