Rewrite the actions page #128

Merged

jcoffey-dev merged 2 commits from phase4/actions into main 2026-10-19 04:17:38 +00:00

Conversation 0 Commits 2 Files Changed 6

+970 -741

jcoffey-dev commented 2026-10-19 04:14:43 +00:00

Owner

Copy link

Copy source

Phase 4, part 3: the actions page. Stacked on #127, so merge that first.

Grouping the actions, building the request, reading the result and checking required inputs move to actions.ts with tests; the page is rewritten around it.

One fix: an action with no options now shows its name while it runs.

After this PR: 84.0%, 12,034 upstream lines left in 47 files, 35 dual headers. Tests: 766 → 775.

Visual check: see #126. It covered all six phase 4 PRs and found no change in the UI.

Typecheck, lint, i18n:check, tests, build and notice check pass locally. No new strings.

Phase 4, part 3: the actions page. Stacked on #127, so merge that first. Grouping the actions, building the request, reading the result and checking required inputs move to `actions.ts` with tests; the page is rewritten around it. One fix: an action with no options now shows its name while it runs. After this PR: 84.0%, 12,034 upstream lines left in 47 files, 35 dual headers. Tests: 766 → 775. **Visual check:** see #126. It covered all six phase 4 PRs and found no change in the UI. Typecheck, lint, i18n:check, tests, build and notice check pass locally. No new strings.

jcoffey-dev added 3 commits 2026-10-19 04:14:43 +00:00

Rewrite the delivery trace page ... b1d833bbe4

The trace's logic is now plain, tested code: trace.ts pairs each step's
start with its outcome, groups steps by delivery attempt, names and
describes them, and holds the trace's states as a reducer; stream.ts runs
the token and the event stream. The page only draws them, unchanged.

Two fixes on the way: "Started Ns ago" counts from the trace's own start
(it carried on from the previous trace before), and cancelling while the
token is still on its way no longer opens the stream afterwards.

Rewrite the setup wizard ... 737eac2279

Its rules are now plain, tested code in wizard.ts: which fields each step
shows, which are required, where the server's refusal lands, and what the
finish screen lists. The wizard itself is rewritten on top of them, with
the same steps, look and update.

Rewrite the actions page ... 39d9062ccf

Its rules are now plain, tested code in actions.ts: grouping actions by
label, what creating one sends (no server results, no untouched secrets),
reading the answer, and which options are required. The page is
rewritten on top of them, with the same look and flow. An action without
options now shows its label while it runs, not its internal name.

jcoffey-dev referenced this pull request 2026-10-19 04:14:52 +00:00

Rewrite live tracing, the trace timeline and the trace detail view #129

jcoffey-dev merged commit e1fca33763 into main 2026-10-19 04:17:38 +00:00

jcoffey-dev deleted branch phase4/actions 2026-10-19 04:17:38 +00:00

jcoffey-dev referenced this pull request 2026-10-19 04:18:38 +00:00

Rewrite the setup wizard #127

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something isn't working

dependencies

Dependency updates

documentation

Docs, README or site copy

duplicate

Already tracked elsewhere

enhancement

New feature or improvement

good first issue

Good for newcomers

help wanted

Contributions welcome

invalid

Not a valid issue

needs info

Waiting on the reporter for details

question

Needs an answer, not a code change

security

Security issue or hardening

translation

New language or translation fixes

wontfix

Will not be worked on

priority

p1

High priority

priority

p2

Medium priority

priority

p3

Low priority

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

jcoffey-dev (John Coffey)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: inbuxa/inbuxa-admin#128