Skip to content

Normalize workflows#24

Merged
hawkinslabdev merged 2 commits into
hawkinslabdev:mainfrom
mvarrieur:pr-4
Apr 29, 2026
Merged

Normalize workflows#24
hawkinslabdev merged 2 commits into
hawkinslabdev:mainfrom
mvarrieur:pr-4

Conversation

@mvarrieur
Copy link
Copy Markdown
Contributor

Summary

  • Refactor workflow trigger evaluation into shared triggers.ts logic so maintenance and date-based rules use normalized trigger handling and reusable notification variable builders.
  • Add workflow preview helpers to compute “next fire” states for the settings page, including measurement-based previews, cooldown/waiting states, and a fix for same-day calendar occurrences.
  • Update workflow UI copy and notification templates to use measurement-neutral wording instead of km-only text, and add coverage for trigger normalization/evaluation plus preview date behavior.

@hawkinslabdev hawkinslabdev self-requested a review April 28, 2026 15:15
@hawkinslabdev hawkinslabdev merged commit b92b019 into hawkinslabdev:main Apr 29, 2026
1 check passed
@hawkinslabdev
Copy link
Copy Markdown
Owner

Awesome! Thanks again @mvarrieur.

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