Repository navigation
feat(app): set and clear repeat rules on reminders - #19
Open
baileywickham wants to merge 1 commit into
Open
baileywickham wants to merge 1 commit into
baileywickham wants to merge 1 commit into
Conversation
`event` reads recurrenceRules but had no way to write them. Add --recurrence <daily|weekly|monthly|yearly> with --recurrence-interval, --recurrence-end | --recurrence-count, --recurrence-days, --recurrence-days-of-month and --recurrence-months to `reminders create` and `update`, plus --clear-recurrence on update, following the LocationOptions pattern. RecurrenceRule gains occurrenceCount so count-limited series round-trip through JSON.
baileywickham
force-pushed
the
feat/reminder-recurrence
branch
from
September 12, 2026 15:11
991a6e7 to
f47f4cd
Compare
baileywickham
marked this pull request as ready for review
September 12, 2026 15:11
|
Code Terrier Review unavailable for this commit: every reviewer model hit an infrastructure fault (upstream provider unavailable, not a code problem) - push a new commit or |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
eventreads a reminder'srecurrenceRulesbut has no way to write them. This adds repeat-rule flags toreminders create/update, following theLocationOptionspattern:--recurrence <daily|weekly|monthly|yearly>, refined by--recurrence-interval,--recurrence-end|--recurrence-count,--recurrence-days(names, 3-letter abbreviations or 1–7 with 1 = Sunday),--recurrence-days-of-month,--recurrence-months;--clear-recurrenceon update.RecurrenceRule.toEKRecurrenceRule()is the inverse ofinit(from:). A rule on a reminder without a due date fails with a clear message instead of an EventKit save error.RecurrenceRulegains an optionaloccurrenceCount, so count-limited series round-trip through JSON. Defaults tonil; existing call sites and payloads are unaffected.Tested with
swift test(new parsing/validation and EventKit round-trip tests) and against a real Reminders store on macOS 26.