-
Notifications
You must be signed in to change notification settings - Fork 104
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
Improvement: Rework handling of timers #476
Conversation
@ksooo, I hope this way the implementation makes sense now. |
Very nice, thanks a bunch. |
1b3bea3
to
d64e996
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also we can't merge this before #475 due to touching strings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all good except one tiny point :)
c2c297b
to
070343c
Compare
Squashed the fixup commits. Will wait for the updated weblate PR now. |
please fix conflicts |
TimerDetails are not used by the App. Do not request them.
Use "Timer" instead of "Geplante Aufnahmen". Listed timers and timer rules will also show reminders/alarms, not only scheduled recordings.
070343c
to
357f75f
Compare
Rebased to master. |
Description
Closes #444.
This PR implements an separate lists for timer and timer rules. This is still work in progress, so I am keeping this as a draft until this settled.
Details:
istimerule
property.Screenshot:
Summary for release notes
Feature: Separate views for timers and timer rules
Improvement: Timer rules show summary, e.g. "Each day from 8:00 PM to 8:15 PM"
Bugfix: Correct German translation around timers