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

Perform additional check for no windows on Amazon Fresh #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mokudekiru
Copy link

Amazon Fresh has a different site layout when no delivery windows are available than Whole Foods (at least in San Francisco - screenshot below). This makes the script reach an error state and results in going back to the beginning of the cart each time, losing out on 50% of potential window opportunities.

My solution is to add one last minute check after checking for windows, specifically for Amazon Fresh. If we do this earlier (alongside the first no window check) we would get false negatives when there are no delivery windows today, but there are some the following days. Not the ideal solution imo, but it avoids the unnecessary error + restart.

Screen Shot 2020-04-12 at 3 18 52 PM

@mark-thompson
Copy link

@mokudekiru Would you happen to have the source (or at least relevant element properties) for that page? I'm working on a platform-independent tool for this and haven't been able to test available slots on Amazon Fresh.

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.

2 participants