Skip to content

Fix repeating timer initial delay - #75

Closed
ipavlidakis wants to merge 1 commit into
developfrom
iliaspavlidakis/fix-repeating-timer-initial-delay
Closed

Fix repeating timer initial delay#75
ipavlidakis wants to merge 1 commit into
developfrom
iliaspavlidakis/fix-repeating-timer-initial-delay

Conversation

@ipavlidakis

Copy link
Copy Markdown
Contributor

🔗 Issue Links

N/A

🎯 Goal

Honor the requested interval before a repeating timer's first fire.

📝 Summary

  • Schedule the first deadline using the TimeInterval value in seconds.
  • Add a regression test that rejects an early first fire.

🛠 Implementation

Uses the native DispatchTime + TimeInterval operation already used by the
one-shot timer. This avoids interpreting a seconds value as milliseconds and
preserves fractional intervals.

🎨 Showcase

N/A

🧪 Manual Testing Notes

Automated timing coverage verifies the behavior. No manual QA is required.

☑️ Contributor Checklist

  • I have signed the Stream CLA (required)
  • This change should be manually QAed
  • Changelog is updated with client-facing changes
  • Changelog is updated with new localization keys
  • New code is covered by unit tests
  • Documentation has been updated in the docs-content repo

@github-actions

Copy link
Copy Markdown
1 Message
📖 Skipping Danger since the Pull Request is classed as Draft/Work In Progress

Generated by 🚫 Danger

@sonarqubecloud

Copy link
Copy Markdown

@ipavlidakis
ipavlidakis deleted the iliaspavlidakis/fix-repeating-timer-initial-delay branch July 27, 2026 12:35
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.

1 participant