-
Notifications
You must be signed in to change notification settings - Fork 48
Updater Support For Custom Respositories #214
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
base: develop_kodi21
Are you sure you want to change the base?
Conversation
|
Hmm, good idea and good effort, thanks! I think it's very overcomplicated, though. It enables custom repositories for people who want to test development on the updater, which is nearly nobody. The most straight forward and unintrusive implementation would be:
AddonSettings are outside of the addon itself (when you long press on the icon, settings) and are only meant for powerusers anyways. This would fit perfectly there. Edit: Additionally, that would avoid having to set anything in the code to activate it |
|
Updated my answer with pointers |
|
Actually, this might need two settings:
Alternatively, this could also be split up into UpdaterCustomBaseDLHost, UpdaterCustomBaseInfoHost (to allow for repositories not on github), CustomUpdaterBranch and CustomUpdaterRepo. You've already implemented parts of that, the step towards a leaner solution should be small. |
|
The thought, and how I’ve used it, was once the settings I’ve implemented are changed to your own repo/branch, it just updates to your forked version if the addon the same way (pushing a version bump in addon.xml). This allowed me to update beyond the updater. I like your method though better and will take a look at that! Good to hear you are okay with putting it in the settings outside the addon, and I really appreciate your pointers! 🙏 |
…x-for-kodi into custom-updater and updated updater to reflect changes
|
Just wanted to follow up on this: |
|
No worries :) We're in the 0.8.0-prerelease stage with a feature freeze, this'll get merged once I start development on 0.8.1, so not time sensitive. |
913c672 to
2ed2f20
Compare
GHI (If applicable): n/a
Description:
Created a new feature to allow using a custom repository & branch as an update source, intending to be used by developers testing out code changes for their fork or branch using the same updater mechanism you have built.
I understand that we wouldn't want general users to accidentally enable this and end up with a broken updater or worse, so this is only available by code changes after the repository has been forked.
I have created a new folder called docs to store wiki pages and wrote a guide for using this for developers to reference, you can also see more detail of how this works. I am pretty unfamiliar with this part, so I am not sure if the structure for this is correct.
I am pretty new to addon development, pushing code to Github, and pretty unfamiliar with your code. I'm by no means a professional at programming (just do it as a hobby), but I tried to add my code within the same design pattern you already had.
I apologize if anything was done incorrectly, let me know and I am happy to take a look!
Checklist: