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

IN_STOCK_WAIT_TIME not working as intended #315

Closed
jesseperrone opened this issue Sep 25, 2020 · 6 comments · Fixed by #325
Closed

IN_STOCK_WAIT_TIME not working as intended #315

jesseperrone opened this issue Sep 25, 2020 · 6 comments · Fixed by #325
Assignees

Comments

@jesseperrone
Copy link

jesseperrone commented Sep 25, 2020

Description

Set IN_STOCK_WAIT_TIME in .env file to several different options including 3600 which should make it so the same link isn't notified more than once an hour but was still getting the same link notification once every 2 minutes.

Screenshot_17

@geman220
Copy link
Contributor

geman220 commented Sep 25, 2020

Works for me, you don't see [${store.name}] Has stock, waiting before trying to lookup again... in the console when stock is detected?

@serg06
Copy link
Contributor

serg06 commented Sep 25, 2020

After pulling the latest code, it doesn't work for me either. I don't see that message, ever.

@geman220
Copy link
Contributor

Are you getting positive stock alerts? The message would only come up if you have a positive hit and it skips that hit again.

@Kommeh
Copy link

Kommeh commented Sep 25, 2020

I get a positive hit and it never skips that item even when it should wait 3600 seconds for my settings. I also do not get that message since earlier code updates.

@jesseperrone
Copy link
Author

jesseperrone commented Sep 26, 2020 via email

@dpgagnon
Copy link

I commented on where the issue is here:
c7a716f#r42727890

jef added a commit that referenced this issue Sep 26, 2020
Fixes #315

Signed-off-by: Jef LeCompte <jeffreylec@gmail.com>
@jef jef closed this as completed in #325 Sep 26, 2020
jef added a commit that referenced this issue Sep 26, 2020
Fixes #315

Signed-off-by: Jef LeCompte <jeffreylec@gmail.com>
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 a pull request may close this issue.

6 participants