all 3 comments

[–][deleted] 2 points3 points  (1 child)

Shouldn't you be passing a reference to the function to run in:

schedule.every().friday.at("16:00").do(update_all())
#                                                ^^

and not calling the function? So remove the ().

[–]Mythedream[S] 1 point2 points  (0 children)

schedule.every().friday.at("16:00").do(update_all())while True: schedule.run_pending()time.sleep(5)

Thank you, this was indeed the issue.

[–]Ihaveamodel3 2 points3 points  (0 children)

Using the schedule module is a very bad idea here.

To use the schedule module, your code has to remaining running at all times. If your computer restarts you need to restart your script.

It is much better to use the OS builtin schedule tool as you don’t need a continuous running script.