-
Notifications
You must be signed in to change notification settings - Fork 15
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
Script is adding every other offer, not each one #2
Comments
@a1ch3mist thanks for reporting! Will take a look at the issue |
@a1ch3mist could you help try out latest version https://github.com/qibinlou/Amex-Offers-Helper/blob/master/index.js ? Also FYI, there are two categories of offer buttons on the offer list page 1) no-action-required offers 2) action-required offers such as redirecting to trycaviar.com to create an account. This script only help users register no-action-required offers. |
It's still missing some. Perhaps you have to delay between each click? AMEX Offers takes a second or two to "register" the click. Tried on an account with 96 offers available. A 2nd run caught the rest of them. I did catch a "Sorry About That!" error quickly on the Web page while I was watching it add, so maybe there are occasional server errors? Is there a 100-limit on the actual added offers as well, or just on the available? |
@a1ch3mist the script waits 2s after each click on Regarding |
Correct, those were offers with the "Add to Card" button. I understand the difference like you said. 2s might not be enough time. I've seen some take longer than that. |
Using Chrome devtools
The script is adding every other offer, not each one. Also seems to occasionally skip more than every other one.
The text was updated successfully, but these errors were encountered: