Skip to content

[Bugfix] Improve initialization of Task Interval#5571

Open
tonhuisman wants to merge 2 commits into
letscontrolit:megafrom
tonhuisman:bugfix/TaskInterval-initialization
Open

[Bugfix] Improve initialization of Task Interval#5571
tonhuisman wants to merge 2 commits into
letscontrolit:megafrom
tonhuisman:bugfix/TaskInterval-initialization

Conversation

@tonhuisman

@tonhuisman tonhuisman commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Resolves #5570

Bugfix:

  • Initialization of the default Task Interval moved to initial task setup, to make it visible immediately after selection of the plugin.

TODO:

  • Testing

@SuksAE

SuksAE commented Jun 29, 2026

Copy link
Copy Markdown

Thanks for the fast response...

tested with P025 (dev.TimerOptional = false): looks good, now the Interval is initialized to 60s - excellent
tested with P001(dev.TimerOptional = true): Interval is initialized to 0s - as before, but - after submitting with Interval to 1s, it can't be set back to 0s - which is different than before. But maybe this is intentional?

regards

@tonhuisman

Copy link
Copy Markdown
Contributor Author

after submitting with Interval to 1s, it can't be set back to 0s - which is different than before. But maybe this is intentional?

Ah, that's a scenario I hadn't tested yet, but I immediately understand why it's happening. Will fix later today.

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.

[BUG] Optional Task Interval does not seem to respect '0' sec.

2 participants