Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

fix(mdUtil): fix throttle() delay check #977

Closed
wants to merge 1 commit into from

Conversation

gkalpak
Copy link
Member

@gkalpak gkalpak commented Dec 18, 2014

Previously, the check was recent - now > delay, but it would never evaluate to true, since now > recent.

@googlebot
Copy link

CLAs look good, thanks!

@gkalpak gkalpak force-pushed the mdUtil-fix-throttle branch 3 times, most recently from 7c0c658 to 5915c50 Compare December 24, 2014 09:43
@ThomasBurleson
Copy link
Contributor

@gkalpak - I will close for next commit. But... it would be great to have a unit test to confirm this. ;-)

@gkalpak gkalpak deleted the mdUtil-fix-throttle branch January 9, 2015 10:54
gkalpak added a commit to gkalpak/material that referenced this pull request Jan 20, 2015
gkalpak added a commit to gkalpak/material that referenced this pull request Jan 20, 2015
@gkalpak
Copy link
Member Author

gkalpak commented Jan 20, 2015

Tests for throttle() added with #1236.

/ping @ThomasBurleson

ThomasBurleson pushed a commit that referenced this pull request Jan 27, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants