feat(android): edit non-recurring events from the detail sheet #73

Merged

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

Conversation 2 Commits 1 Files changed 3 +173 -45

nalum commented 2026-08-14 15:26:29 +00:00

Owner

Copy link

Fourth part of #61: Android events become editable, matching web #71.

EventService.Update was unreachable on Android — an event was create-only. Adds repo.updateEvent and an Edit action on the event sheet that reopens the create form prefilled from the occurrence row (toEventDraft reverses it back into the form fields) and saves through updateEvent. Gated on the permission matrix (can("EventService","Update")).

Scoped to non-recurring events, mirroring the web: the Edit action is withheld from recurring series and projected occurrences, and the repeat control is hidden in edit mode.

Remaining in #61: recurring event edit (occurrence-vs-series) on both surfaces — the next PR.

Verified: make check passes (:app/:data compile, Go tests, web build, i18n — strings shared with #71, no new ones).

Part of #61.

🤖 Generated with Claude Code

Fourth part of #61: Android events become editable, matching web #71. `EventService.Update` was unreachable on Android — an event was create-only. Adds `repo.updateEvent` and an **Edit** action on the event sheet that reopens the create form prefilled from the occurrence row (`toEventDraft` reverses it back into the form fields) and saves through `updateEvent`. Gated on the permission matrix (`can("EventService","Update")`). Scoped to non-recurring events, mirroring the web: the Edit action is withheld from recurring series and projected occurrences, and the repeat control is hidden in edit mode. Remaining in #61: recurring event edit (occurrence-vs-series) on both surfaces — the next PR. Verified: `make check` passes (`:app`/`:data` compile, Go tests, web build, i18n — strings shared with #71, no new ones). Part of #61. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

nalum added 1 commit 2026-08-14 15:26:29 +00:00

feat(android): edit non-recurring events from the detail sheet ... e19d7cfc79

Mirrors the web: EventService.Update was unreachable on Android, so an
event was create-only. Add repo.updateEvent and an Edit action on the
event sheet that reopens the create form prefilled from the occurrence
row and saves through it, gated on the permission matrix
(EventService.Update). Non-recurring events only, matching the web; the
repeat control is hidden in edit mode. Part of #61.

nalum added the

enhancement

android

labels 2026-08-14 15:26:29 +00:00

forgejo-actions commented 2026-08-14 15:29:03 +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 c01afe6086}

<!-- 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 c01afe60861a74924ddb0fddb53ad7f74c2f71a5}

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

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

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

Backlog ordering (tracking) #62

nalum force-pushed feat/android-event-edit from e19d7cfc79 to c01afe6086 2026-08-14 16:03:15 +00:00 Compare

forgejo-actions commented 2026-08-14 16:46:09 +00:00

Copy link

Android test report

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

_{Updated by the android workflow · commit c01afe6086}

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

nalum changed target branch from feat/android-affordances-from-matrix to main 2026-08-14 16:56:17 +00:00

nalum merged commit c01afe6086 into main 2026-08-14 16:56:24 +00:00

nalum deleted branch feat/android-event-edit 2026-08-14 16:56:24 +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!73

No description provided.