-
Notifications
You must be signed in to change notification settings - Fork 149
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
Use pyttsx3 text-to-speech module instead of os.system('say "..."') for cross-platform compatibility #32
base: master
Are you sure you want to change the base?
Conversation
…ay which only works with Macs
Added 1-hour delivery windows and 2-hour delivery windows
Fixes for 1-2 day here
…atform pyttsx3 TTS module
Thanks a lot for getting this out. I've added a link to your PR in the Readme, meanwhile I try to test this. Thank you! |
amazon_fresh_delivery_slot_chrome.py
Outdated
@@ -48,7 +54,7 @@ def getWFSlot(productUrl): | |||
open_slots = soup.find('div', class_ ='orderSlotExists').text() | |||
if open_slots != "false": | |||
print('SLOTS OPEN!') | |||
os.system('say "Slots for delivery opened!"') | |||
sayIt("Slots for delivery opened!" |
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.
sayIt("Slots for delivery opened!")
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.
Fixed
This pull request uses pyttsx3 TTS (text-to-speech) module instead of os.system('say "..."') for cross-platform compatibility - allowing scripts to work on Mac, Windows, and Linux without changes.
This should also allow removal of any Windows-specific scripts in the repo, reducing maintenance/upkeep needed.