feat: edit recurring events, this-one or the whole series #74

Merged

nalum merged 1 commit from feat/recurring-event-edit into main 2026-08-14 16:56:30 +00:00

Conversation 2 Commits 1 Files changed 18 +239 -24

nalum commented 2026-08-14 15:35:03 +00:00

Owner

Copy link

Final part of #61: recurring events become editable on both surfaces, with a this-one-or-the-series choice. Closes the event-editing half of #61.

Non-recurring events already edit in place (#71 web, #73 Android). This adds editing for a repeating occurrence: the event sheet offers a scope choice —

• This event: SplitOccurrence splits the occurrence into a one-off child, then the form edits that child. Only that instance changes.
• The whole series: reads the parent (events.read) and edits it; the change lands on every occurrence. The repeat rule itself stays fixed (editing recurrence is still a separate flow).

Web: EventDetail gains the scope picker; Calendar wires editOne (split) and editSeries (read).

Android: EventSheet gains the picker; MoreRepositories gains splitForEdit/eventForEdit, mapping the returned event into the row the form prefills from.

Verified: make check passes (web tsc/build, :app/:data compile, Go tests, web + Android i18n catalogs synced for the two new strings).

Part of #61 (the last piece).

🤖 Generated with Claude Code

Final part of #61: recurring events become editable on both surfaces, with a this-one-or-the-series choice. Closes the event-editing half of #61. Non-recurring events already edit in place (#71 web, #73 Android). This adds editing for a repeating occurrence: the event sheet offers a scope choice — - **This event**: `SplitOccurrence` splits the occurrence into a one-off child, then the form edits that child. Only that instance changes. - **The whole series**: reads the parent (`events.read`) and edits it; the change lands on every occurrence. The repeat rule itself stays fixed (editing recurrence is still a separate flow). Web: `EventDetail` gains the scope picker; `Calendar` wires `editOne` (split) and `editSeries` (read). Android: `EventSheet` gains the picker; `MoreRepositories` gains `splitForEdit`/`eventForEdit`, mapping the returned event into the row the form prefills from. Verified: `make check` passes (web tsc/build, `:app`/`:data` compile, Go tests, web + Android i18n catalogs synced for the two new strings). Part of #61 (the last piece). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

nalum added 1 commit 2026-08-14 15:35:03 +00:00

feat: edit recurring events, this-one or the whole series ... 3817392a99

Non-recurring events already edit in place; recurring ones offered no
edit. Add a scope choice on the event sheet for a repeating occurrence:
'This event' splits the occurrence off (SplitOccurrence) and edits that
one child; 'The whole series' reads the parent and edits it, changing
every occurrence (the repeat rule itself stays fixed — that is still a
separate flow). Both surfaces. Closes the event-editing half of #61.

nalum added the

enhancement

android

web

labels 2026-08-14 15:35:03 +00:00

nalum referenced this pull request 2026-08-14 15:35:37 +00:00

Backlog ordering (tracking) #62

forgejo-actions commented 2026-08-14 15:38:08 +00:00

Copy link

Test report

Suite	Tests	Result	Skipped
Unit	1330	✅ pass	1
Integration	83	✅ pass	—

Coverage: 27.7%

_{Updated by the check workflow · commit 710402a795}

<!-- ci-test-report --> ## Test report | Suite | Tests | Result | Skipped | | --- | --: | --- | --: | | Unit | 1330 | ✅ pass | 1 | | Integration | 83 | ✅ pass | — | **Coverage:** 27.7% _{Updated by the check workflow · commit 710402a795de277acb19a83208105dd69aca5503}

nalum force-pushed feat/recurring-event-edit from 3817392a99 to 710402a795 2026-08-14 16:03:16 +00:00 Compare

forgejo-actions commented 2026-08-14 16:47:19 +00:00

Copy link

Android test report

Suite	Tests	Result	Skipped
Unit (debug)	31	✅ pass	0

_{Updated by the android workflow · commit 710402a795}

<!-- android-test-report --> ## Android test report | Suite | Tests | Result | Skipped | | --- | --: | --- | --: | | Unit (debug) | 31 | ✅ pass | 0 | _{Updated by the android workflow · commit 710402a795de277acb19a83208105dd69aca5503}

nalum changed target branch from feat/android-event-edit to main 2026-08-14 16:56:24 +00:00

nalum merged commit 710402a795 into main 2026-08-14 16:56:30 +00:00

nalum deleted branch feat/recurring-event-edit 2026-08-14 16:56:30 +00:00

nalum referenced this pull request 2026-08-14 16:58:25 +00:00

Drive UI create/manage/edit affordances off the permission matrix (web + Android) #61

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

adr

Needs or introduces an Architecture Decision Record

  

android

affects android app

  

area/calendar

The calendar surfaces: month, week, agenda, day sheet, events

  

area/design-system

Tokens, shared components and the row grammar both surfaces use

  

area/i18n

Strings, msgids and translation across surfaces

  

area/jobs

The jobs surface: board, card, sheet, create

  

area/offline

The offline write queue and sync behaviour

  

area/server

Server behaviour: RPCs, guards, domain rules, proto

  

area/testing

Conformance fixtures, goldens and test harnesses

  

bug

Something is not working

  

ci

CI and release pipeline

  

duplicate

This issue or pull request already exists

  

enhancement

New feature

  

help wanted

Need some help

  

invalid

Something is wrong

  

notifications

Push and notification system

  

question

More information is needed

  

reliability

Robustness, failure modes, resilience

  

security

Security finding or hardening work

  

severity/low

Low severity

  

severity/medium

Medium severity

  

tracking

Plan / tracking issues

  

web

affects web app

  

wontfix

This won't be fixed

No labels

adr

android

area/calendar

area/design-system

area/i18n

area/jobs

area/offline

area/server

area/testing

bug

ci

duplicate

enhancement

help wanted

invalid

notifications

question

reliability

security

severity/low

severity/medium

tracking

web

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

eagraiclainne/app!74

No description provided.