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

Add episodes to sleep timer #4754

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

asdoi
Copy link
Contributor

@asdoi asdoi commented Dec 11, 2020

Continuation of #4535

Closes #4535
Closes #2146

This adds the basic functionality to the sleep timer to stop after the end of the xth episode.
I think this is ready for a short review, however there are two "bugs" with the checkboxes:

  1. Vibrate shortly before end: The phone now vibrates when the last episode starts
  2. Auto-enable: After the playback ended the auto-enable option is triggered and reenables the sleep timer, but nothing is playing.

@keunes
Copy link
Member

keunes commented Dec 11, 2020

Auto-enable: After the playback ended the auto-enable option is triggered and reenables the sleep timer, but nothing is playing.

What do you mean with this one? Isn't it normal that nothing is playing after playback ended? ;)

@asdoi
Copy link
Contributor Author

asdoi commented Dec 11, 2020

I see that this was not the best phrasing. I meant, that after the sleep timer ended the episode and stopped itself, the auto-enable option re-enables it. So that the playback stopped correctly, but the sleep timer is turned on again.

@keunes
Copy link
Member

keunes commented Dec 22, 2020

Sorry, seeing your reply only now.

after the sleep timer ended the episode and stopped itself, the auto-enable option re-enables it. So […] the sleep timer is turned on again

This also seems like expected behaviour, if auto-enable is activated.

The only issue is that the effect is that the behaviour effectively is the same as when continuous playback is disabled. So I can imagine auto-enable is disabled & deactivated when 'at end of episode' is active. At least this potentially confused users.

@antennapod-bot
Copy link

This pull request has been mentioned on AntennaPod Forum. There might be relevant details there:

https://forum.antennapod.org/t/set-the-off-timer-at-the-end-of-the-current-episode/689/2

@keunes
Copy link
Member

keunes commented Mar 1, 2021

@asdoi Would you still be interested in this PR? Would be great to get this added in AntennaPod :)

@ByteHamster
Copy link
Member

Sorry, it's actually my turn. I did not have a look at it yet.

@antennapod-bot
Copy link

This pull request has been mentioned on AntennaPod Forum. There might be relevant details there:

https://forum.antennapod.org/t/sleep-timer-until-end-of-episode/1214/2

@rubo77
Copy link

rubo77 commented Oct 9, 2022

This would be really great, to have this in the next release.

What is the status?

@Fruchtzwerg94
Copy link

Same question from my side. Any plans to being this in soon?

@AntennaPod AntennaPod locked and limited conversation to collaborators Oct 27, 2022
@ByteHamster ByteHamster added the Needs: Decision Proposal and most arguments are clear, but needs a verdict. label Jul 16, 2023
@keunes
Copy link
Member

keunes commented Aug 3, 2023

Hi @ByteHamster,
I see you added the 'Needs decision' label - what kind of decision did you have in mind? (My position on this functionality: #2146 (comment))

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Needs: Decision Proposal and most arguments are clear, but needs a verdict.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sleep Timer: Stop at end of episode
6 participants