-
Notifications
You must be signed in to change notification settings - Fork 34
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
Control access to recordings and timers #260
Conversation
ed5fb01
to
5aaf001
Compare
@phunkyfish could you please re-review. I revamped the original commit to one setting rather than two. I found a few hardcoded English string in instance-settings.xml that I was lax on updating. I also fixed an outstanding issue with missing timer types that had not been added to Kodi. |
Don’t want to block this PR. only out of curiosity: does nextpvr backend not have a multi user concept? Different users can log in with different credentials, having their own set of permissions/config? For me, this would be a clean way to implement what the user requested in the forum. Nothing that should be done by the addon, imo, if the backend offers multi user support. |
No the backend doesn't support multi-user. If I had access to the backend source this is probably the first thing that I would change. As an example having one resume point for a recording can cause conflicts, which is why I added an option to store the resume point locally. That all being said over the almost 20 years I have been involved with the project there have been very few calls to add multi-user support. I think users like the simplicity of installing NextPVR and most wouldn't be bothered to use the feature anyway. |
Provide options to access Live TV only (no recording or timer access) or to disable timers only, disable recordings and disable recordings delete
NextPVR allows timeslot and also episode timers, pvr.nextpvr didn't support all episode single channel timers. Legacy backend code was defaulting the daily timeslot recording to an all episode recording. Added logic for backend type 2 and type 3 timers to handle these situations.
Provide options to access Live TV only (no recording or timer access) or to disable timers only.
Add new timer type single channel all episodes (filtered for new and duplicate at the backend) and Daily Timeslot recording.