-
-
Notifications
You must be signed in to change notification settings - Fork 429
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
Exception in JsonAddonService / marketplace bundle at startup #2687
Comments
@J-N-K : looks like you are the one who contributed this class. |
Why is that property a String in your setup?
It's a boolean for me. |
You are talking about "remote" property ? |
In my file userdata/config/org/openhab/addons.config, the generated value is like that:
In my file conf/services/addons.cfg, the value is set like that:
It was updated because the default provided with the distribution is like that:
So, I just uncommented the line and replace true by false. And this is like that since several years. |
I don't use config files. Setting it in the UI makes it like I showed. Seems someone has to decide what's correct and first align the setting. If needed, I can adjust the code then. |
Maybe the code can accept boolean and string? |
Look how this config parameter is correctly handled elsewhere: Line 180 in cabb3f7
|
Looks like an ugly workaround for the correct solution. But ok. |
last snapshot 2694, at startup of OH:
The text was updated successfully, but these errors were encountered: