-
Notifications
You must be signed in to change notification settings - Fork 42
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
Suggest to use new settings.xml layout for v18 and up #127
Comments
I don't know if we have a page for the new settings for add-ons. If we have one, I would be interested to check whether it contains all info for add-on devs to migrate. |
@MartijnKaijser @basrieter are we talking about https://kodi.wiki/view/Add-on_settings? |
Do we need to check something specific within the file or do we just want to check whether its present or not? |
Correct: So you need to detect some specific things that identify it as old or new layout |
A better approach would be to create schemas for all elements in the settings similarly to what we have done for addon xml and just validate that. We have yet to document the new settings format too. Depending on time constraints I may have a look at doing it. |
Since several version we support a new settings.xml format which is the same as Kodi itself uses.
As most bugs should have been solved by now we should suggest developers to start using this new layout.
I'm sure there is already some wiki page or forum post explaining how the new layout works
The text was updated successfully, but these errors were encountered: