Skip to content

Develop - #13

Merged
EarMaster merged 5 commits into
mainfrom
develop
Sep 8, 2026
Merged

Develop#13
EarMaster merged 5 commits into
mainfrom
develop

Conversation

@EarMaster

Copy link
Copy Markdown
Owner

No description provided.

EarMaster and others added 5 commits August 27, 2026 14:21
Play's Edits.commit rejects the parameter outright for this app —
"Changes are sent for review automatically. The query parameter
changesNotSentForReview must not be set." — so v1.2.2's deploy failed
at the commit step with the AAB already uploaded. v1.2.1 committed fine
without it; the flag was added pre-emptively for the opposite error,
which this app does not produce.

The two states are mutually exclusive, so only set it if Play starts
refusing the other way. A failed commit abandons the edit, so v1.2.2
can be re-uploaded under the same versionCode.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…reenshots]

The tab showed favourites in the order they were saved and nothing said
so; the only way to move one was two entries in its own ⋮ menu, one step
at a time. A sort action now sits beside the layout switch, with the
current order ticked. Sorting never writes sortIndex, so switching back
to the hand-made arrangement brings it out as it was — and move up/down
are offered only in that order, where they can be seen to work.

The marker re-records the goldens on CI: the three favourites pictures
gained a top bar icon, and favourites_sorted_* is new.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@EarMaster
EarMaster merged commit 1aeb577 into main Sep 8, 2026
10 checks passed
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.

1 participant