Skip to content
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

Inconistent approach to allowing re-record #930

Open
white-haired-uncle opened this issue Jul 14, 2024 · 3 comments
Open

Inconistent approach to allowing re-record #930

white-haired-uncle opened this issue Jul 14, 2024 · 3 comments

Comments

@white-haired-uncle
Copy link

Is your feature request related to a problem? Please describe.

If you want to allow a recording to re-record, look under Recording Options (not Storage Options).

If you want to allow several recordings to re-record, it's easier to assign them to a play list, and then choose re-record under Storage Options (there is no Recording Options). Kind of confusing. Also, I'm pretty sure Allow Re-record is worded differently

Also, as soon as you do so the playlist seems to disappear, or at least all the check marks go away. What if I also wanted to disable Auto Expire on the same group (or something, that may be a dumb example).

Describe the solution you'd like

I would create Recording Options under Playlist Options. I think. But it's consistency that's important.

Describe alternatives you've considered

It works as it is today, it's just inconsistent.

@white-haired-uncle
Copy link
Author

Hate to pile on here, but if you set a recording to allow re-record and then go to Recording Options, there is an option "Allow Episode to Re-Record" (or something very close to this), which should be "Disable Re-Record" (or something like that). I'm not sure what the selection actually does, whether it's a functionality problem or just a labelling problem.

@bennettpeter
Copy link
Member

Regarding disable re-record. The "allow re-record" does not change to "disable re-record" if re-record is already allowed. Allowing re-record again goes through the motions of allowing re-record, which does nothing if re-record is already allowed.

Allowing re-record works by deleting recorded history for the episode. Disabling re-record would require creating history for the episode. Currently this can be done in upcoming recordings by selecting "Never record this episode". However that can only be done if the episode is upcoming.

@white-haired-uncle
Copy link
Author

Okay, thanks, that makes some sense. I'm a bit surprised re-creating the history couldn't be done from recorded, but not worried enough to look into it.

However, once a recording is set to allow re-record, the option probably shouldn't show up in Recording Options. It's confusing, IMO, making the user suspect they did something wrong the first time. This user, at least.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants