-
-
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
Fix addon name localisation before installation #3908
Conversation
Signed-off-by: Mark Herwege <[email protected]>
@andrewfg FYI |
Signed-off-by: Mark Herwege <[email protected]>
@mherwege / @kaikreuzer shall I do my usual full build clean install test before you merge it? i.e. tomorrow. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, let's have a try with this.
@kaikreuzer / @mherwege all Ok; it works on my full build. |
@jimtng Indeed, I think this should be reverted. I don't think there is any harm in having the NA treatment back, and that keeps us on the safe side. It might be needed when loading if all providers are not available at the same time. I am not sure about that, so prefer keeping it. If we revert this, is there a need to update the developer documentation to point out the name and description are required and they should be provided in full in the addon.xml? Reviewer instructions? |
This one needs to be updated: |
@mherwege I'll prepare a PR to revert this |
This reverts commit 708a954 but with some modifications Signed-off-by: Jimmy Tanagra <[email protected]>
Fixes openhab/openhab-webui#2204
Replaces #3905
The issue is that no addon info is available for distribution addons before installation. The only thing available is the feature description and the addon endpoint gets it from the addon data, not the addonInfo data. So a merge of addonInfo is not even happening.
This change introduces a flag in AddonInfo indicating if it is the master info or not. When the info is coming from the addon repository scan, it is flagged to not be the master data and name and description will be ignored (because localization info is not available). Addon services will check if the addon info is master info, and only then update name and description.