Skip to content
This repository was archived by the owner on Jun 23, 2026. It is now read-only.

Watching startTimeAttr and restarting timer if changed#71

Merged
siddii merged 1 commit into
siddii:masterfrom
leamagic:master
Dec 18, 2014
Merged

Watching startTimeAttr and restarting timer if changed#71
siddii merged 1 commit into
siddii:masterfrom
leamagic:master

Conversation

@leamagic

@leamagic leamagic commented May 6, 2014

Copy link
Copy Markdown
Contributor

If the timer is running and the startTimeAttr is changed, the timer will be restarted.

@siddii

siddii commented May 6, 2014

Copy link
Copy Markdown
Owner

Interesting... never thought of this scenario. Thanks for the PR 👍

Can you please add some test for this change?

@alduro

alduro commented Dec 18, 2014

Copy link
Copy Markdown

I do need this as well. Related to issue #139.

@siddii

siddii commented Dec 18, 2014

Copy link
Copy Markdown
Owner

@alduro I am merging this now. Since there is not enough tests around this. Can you verify if it didn't break anything?

siddii added a commit that referenced this pull request Dec 18, 2014
Watching startTimeAttr and restarting timer if changed
@siddii siddii merged commit 252538f into siddii:master Dec 18, 2014
@mgoldsborough

Copy link
Copy Markdown

Great add. Thanks, @alduro!

@siddii, any timetable on a new release to bower?

@siddii

siddii commented Feb 25, 2015

Copy link
Copy Markdown
Owner

Here is a new release - https://github.com/siddii/angular-timer/releases/tag/v1.2.1

@mgoldsborough

Copy link
Copy Markdown

Awesome. Thanks, @siddii!

@mgoldsborough

Copy link
Copy Markdown

@siddii was the dist directory rebuilt for v1.2.1? Still looks like v1.2.0.

@siddii

siddii commented Feb 25, 2015

Copy link
Copy Markdown
Owner

Good point, I'll check

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