Reopened- GUI Configuration: Created config.schema.json #13
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Created config.schema.json which would allow configuration via GUI in Config-UI-X. Also fixed the license listed in the package.json currently throws an SPDX License error on install due to incorrect ID. All configuration settings can be set via the GUI except
excluded_attributes
andexcluded_capabilities
. If those were to be configured via GUI it would require a change in how users configure them in the config.json. See here for necessary changes . It would be possible put this would break backwards compatibility with previousconfig.json
version. Another breaking change that would make GUI configuration simpler would be reworking how the boolean values in theconfig.json
work. If GUI configuration is used boolean values will be put intoconfig.json
no matter if they are the same as the defaults the plugin uses.I also moved the changelog from the readme to a separate file. This makes the readme a little bit more readable and this way the changelog can be read in to Homebridge UI.
This PR has no breaking changes.
These breaking changes are just being discussed for possibly another PR later to allow better configuration via GUI.
RE: Dependencies from last PR, Winston log rotate has been deprecate.Tried switching to https://github.com/winstonjs/winston-daily-rotate-file however that seemed to break some stuff. I think it would be best to remove the external log and integrate solely into Homebridge's log rather than generating an extraneous log file.