-
Notifications
You must be signed in to change notification settings - Fork 26
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
Support configuration as script arguments #7
Conversation
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.
Jazak Allah these are great enhancements. Thank you for taking the time to add these.
I left a few suggestions. One is a bug fix and the rest are minor updates. If you can make those updates that would be great. I'll merge once done.
Thanks again!
Co-authored-by: Abrar Chaudhry <2893677+achaudhry@users.noreply.github.com>
Co-authored-by: Abrar Chaudhry <2893677+achaudhry@users.noreply.github.com>
Jazana Wa'eyakom, I have addressed all of your comments. Please let me know if anything more is needed to merge this. |
I don't know if this would be the right spot to ask but let me know if not I'll remove this! I am just playing with the praytimes.py and updateazantimer.py and instead of playing the audio on a pi, I am trying to run this in my local mac with the help of crontab but what is confusing me is how could I work around this command that is used for omxplayer to run the audio. Also, it seems as my jobs gets created on my local machine but the audio file never really plays. Any tips or suggestions would be appreciated. |
@Zamy97 that's off topic, but my answer is that for macOS the preferred way to run timed jobs is launchd. Although cron is still supported, but it is not recommended, and I couldn't get it to run on my Mac. |
@achaudhry let me know if you have questions so far to be able to merge. |
@hammady sorry man, been extremely busy over the last few days but I haven't forgotten about this I promise :) Will look shortly. Thanks |
Thanks @achaudhry, now #10 :) |
--lat 20.23 --lng 50.552 --method Egypt
. These settings get persisted automatically so that later runs do not have to receive any arguments--fajr-azaan-volume 1200
and--azaan-volume 1800