Skip to content

Conversation

@working-name
Copy link
Contributor

Let's not send people to an ad-infested abandoned domain name :)

Let's not send people to an ad-infested abandoned domain name :)
Copy link
Owner

@walmik walmik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you @working-name !

@walmik walmik merged commit 8147b4b into walmik:master Jul 10, 2020
mohitYogi added a commit to mohitYogi/timer.jquery that referenced this pull request Dec 5, 2020
* Updated timer to support continuous run (#1) (walmik#92)

* Updated timer to support continuous run (#1)

* Updated intervalHandler to support continuous run

intervalHandler is updated in order to support continuous timer run, even after duration is completed

* Flag for single call to callback

Flag for single call to callback when timer completes in continuous run.

* Update utils.js

* support continuous Timer run

1. Added stopTimerOnDuration in config to run timer continuously.
2. Updated intervalHandler to support continuous run
3. Added Flag isCallbackCalled to stop multiple calls to callback after reaching the duration in continuous run.

* Updates after review.

1. Removed stopTimerOnDuration flag (repeat flag do the same.)
2. Removed isCallbackCalled flag, as suggested after commit review.

* Update after Review

Removed flag isCallbackCalled. (Not required).

* Updates after review

1. Removed stopTimerOnDuration flag (repeat flag do the same.)
2. Removed isCallbackCalled flag, as suggested after commit review.

* Updates after Review

1. Removed stopTimerOnDuration flag (repeat flag do the same.)
2. Removed isCallbackCalled flag, as suggested after commit review.

* Patch version

New rule for checking timer duration completed.

* Update timer.jquery.js

* Semicolon & Comment update

Added missing semi colon. Updated Comment.

* Removed & Updated Comment

* Update README.md (walmik#99)

Let's not send people to an ad-infested abandoned domain name :)

Co-authored-by: Alan <my@working.name>
@working-name working-name deleted the patch-1 branch August 12, 2023 04:42
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