ioBroker / ioBroker.knx

connect KNX via eibd
http://www.iobroker.org
Other
50 stars 32 forks source link

Adapt io-package.json to show license information #310

Closed mcm1957 closed 8 months ago

mcm1957 commented 9 months ago

As this adapter requires a paid license at least for som eusage varainats, please replace 'license' by 'licenseInformation' as soon as possible.

For details see: https://github.com/ioBroker/ioBroker.js-controller/blob/7bcbbdcceee4d1fe7411588bc1b791da29e08747/schemas/io-package.json#L892

chefkoch009 commented 8 months ago

fixed in 2.0.26

foxriver76 commented 8 months ago

Hi Kay, can you maybe adapt the link to link to e.g. https://github.com/ioBroker/ioBroker.knx?tab=readme-ov-file#license-requirements so there users can get the information that they only need license for more than 500 datapoints and where they can buy license. Sorry if this was not clear until now. Also feel free to set licenseInformation.license to CC BY-NC 4.0 so admin can show the correct license on installation.

mcm1957 commented 8 months ago

Sorry, but

a) "license" is missing within licenseInformation. b) "link" must point to an information stating two to buy a license, infomation about prizes and in this case of "limited" the limitations of free version. (Sorry, the descripbtion of link at schema is still misleading, the link is definitly not meant as a link to the license specification)

` "licenseInformation": { "license":"CC BY-NC 4.0" "link":"link-to-information ", "type": "limited" }

` NOTE: a new release is NOT required, a fix at github repo is sufficient as the Repobuilder will pick up the info from there anyway.

Please reopen issue if you do not have time to fix it today

OK: Foxriver was faster :-)