Skip to content

docs: Phase 14 merged, and a rule from its last bug - #31

Merged
AndyScherzinger merged 2 commits into
mainfrom
docs/phase-14-follow-ups
Oct 5, 2026
Merged

AndyScherzinger merged 2 commits into
mainfrom
docs/phase-14-follow-ups

Conversation

@AndyScherzinger

Copy link
Copy Markdown
Contributor

Summary

Two follow-ups to Phase 14 (#30), documentation only:

  • docs/14-status.md moves Phase 14 among what is built (Phase 14: feeds and everyday tools #30, 2026-10-05); nothing is in progress until the next phase starts.
  • AGENTS.md gains a rule under "Code that tends to go wrong": a row in a keyed lazy list that can leave the list and come back holds no rememberSaveable state such as rememberSwipeToDismissBoxState. The list restores that state under the same key, so the row returns as it left. This is the bug where a feed put back by Undo in Edit feeds came back swiped away and was removed again; the review pass missed it and a device test found it.

Test plan

Documentation only; no code or resources change. CI's checks cover the rest.

Checklist

  • One concern; split if it approaches a thousand changed lines
  • Tests for every non-trivial branch; fixtures from the dev instance where the server is involved
  • detekt ktlintCheck lint alohaArchitectureCheck green, no baseline grown
  • Screenshots re-recorded only where the UI change is intended
  • Accessibility: labels, 48 dp targets, headings and paneTitle on new screens, 200 % font preview
  • Strings in strings.xml with translator comments
  • No new exported component, permission or dependency without a line here explaining it
  • docs/ updated where behaviour changed
  • AI tools were used for this contribution (commits carry Assisted-by:)

Status moves Phase 14 among what is built, with its pull request and
date, and nothing is in progress until the next phase starts.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
A removed feed put back by Undo came back swiped away and was removed
again: its swipe state was saveable, and the lazy list restored it under
the same key. The review pass did not catch it; a device test did. The
rule joins the list of code that tends to go wrong.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@AndyScherzinger AndyScherzinger added this to the 1.0.0 milestone Oct 5, 2026
@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown

📱 QA build

Download aloha-qa-pr31.apk
QR code Open the QR code for this download
Commit a0809eb
Version 0.1.0 (1)
Available until 5 days after this build

The QA build installs alongside any other Aloha build, so you can keep using your existing install while testing, and it updates an earlier QA build in place.

Downloading the file requires a GitHub account, so open this link on the device you want to test on, or transfer the APK to it.

@AndyScherzinger
AndyScherzinger merged commit 8f78f87 into main Oct 5, 2026
@AndyScherzinger
AndyScherzinger deleted the docs/phase-14-follow-ups branch October 5, 2026 19:57
@AndyScherzinger AndyScherzinger added documentation Improvements or additions to documentation AI assisted Commits carry an Assisted-by trailer labels Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI assisted Commits carry an Assisted-by trailer documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant