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

scheduled update feature [WIP] [request feedback] #158

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

thigg
Copy link
Contributor

@thigg thigg commented Nov 2, 2020

Added a systemd timer which can be used to automatically refresh the podcasts.
The timer can be configured to a specific time of the day.

  • verify refresh is working properly
  • add option to disable timer
  • make label above button in settings display a nicer representation of the set time
  • check if harbour actually allows timers
  • find a way to refresh the gui once its done. (dbus?)
  • bring python tests into the toolchain
  • make sure there are no conflicts if manual sync is triggered during scheduled and vice versa

fixes #156

@thigg
Copy link
Contributor Author

thigg commented Nov 2, 2020

@sfbg @Keeper-of-the-Keys what do you think?

@Keeper-of-the-Keys
Copy link
Contributor

Looks interesting (just went over code for now), several notes:

  1. Though personally I have unlimited data so I don't care what connection I am on other users don't so ideally an automated service would have some way to limit connection type (this technique could also potentially resolve [Missing Feature] Limit the network bearer #10 but not in the same PR please).
  2. I noticed several hardcoded paths these should be avoided if at all possible, depending on how old a Jolla phone install is it will either be running as nemo or as defaultuser.
  3. Knowing whether harbour allows is important because otherwise this will need to be included in the harbour compatibility patch.
  4. There probably also needs to be a notification of some sorts, personally I like to check how many fresh episodes I have before triggering a refresh so that I know whether this number got changed by the refresh and whether or not to look for new episodes.

(As for point 1 I realize that there is a setting to turn this on and off)

@thigg
Copy link
Contributor Author

thigg commented Nov 6, 2020

Hi,

  1. but I think it is a good idea to either add a hint that scheduled updates will ignore the setting from Config option to allow downloads via wifi only #155, or we need to make the setting available to the service.
  2. a I forgot about that homepath... its relative now.
  3. How do we find it out?
  4. added the notification. That meant, that the pythonscript needs access to a translation mechanism, thats why we have a very ugly gettext stunt now in there. I wasnt able to find a better solution yet.

@thigg
Copy link
Contributor Author

thigg commented Dec 29, 2020

systemd-timers are not allowed yet in jollas store, but they are discussed e.g. here and here

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

Successfully merging this pull request may close these issues.

[feature request] scheduled refresh
2 participants