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

Don't acknowledge offer change if unknown or glitched #110

Merged
merged 2 commits into from Mar 8, 2016
Merged

Don't acknowledge offer change if unknown or glitched #110

merged 2 commits into from Mar 8, 2016

Conversation

jlao4394
Copy link
Contributor

@jlao4394 jlao4394 commented Mar 5, 2016

Unknown offers will not emit "unknownOfferSent" because saved offer state will be set even if delay is not met.

Glitched offers will not emit "sentOfferChanged" because saved offer state is updated before another poll.

someazndude added 2 commits March 5, 2016 12:31
Unknown offers will not emit "unknownOfferSent" because saved offer state will be set even if delay is not met.

Glitched offers will not emit "sentOfferChanged" because saved offer state is updated before another poll.
DoctorMcKay added a commit that referenced this pull request Mar 8, 2016
Don't acknowledge offer change if unknown or glitched
@DoctorMcKay DoctorMcKay merged commit c25a0fb into DoctorMcKay:master Mar 8, 2016
@DoctorMcKay
Copy link
Owner

👍 Thanks. Live in v1.20.1.

@jlao4394 jlao4394 deleted the patch-1 branch March 24, 2016 22:51
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.

None yet

2 participants