NEC-170: Add the missing screenshots to the user guides #236

Merged
nectenda-agent merged 14 commits from worktree-nec-170-add-the-missing-screenshots-to-the-user into main 2026-09-30 21:50:56 +01:00
Collaborator

Task: NEC-170 — https://projectron.nerchure.com/tasks/178

Every user guide now shows the thing each step asks the reader to find: 58 new screenshots and 18 retaken, all from one reproducible journey in packages/e2e/docs-shots.mts through two real vaults, most cropped to a single element. What cannot be reached for real is marked STAGED in the script and in each guide's brief. scripts/guide-labels.test.mjs now requires every guide image to come from docs-shots and none to be orphaned.

Capturing them for real found six defects in shipped code, each fixed here with the user's approval, each with its own card, commit and test (mutation-checked both ways):

  • the sign-in code box was a sliver (NEC-177);
  • the skipped-attachment Download on this device button was removed as soon as it was drawn (NEC-178);
  • an owner never saw Replace… or Turn off… on the share link (NEC-179);
  • the shard's CORS refused PATCH, so share link on/off, role changes and folder names failed from the plugin (NEC-180);
  • a plugin refused as too old showed Connected (NEC-181);
  • a device refused by one organisation showed Connected instead of Device limit reached (NEC-182).

Spec: docs/changes/NEC-170-guide-screenshots/spec.md

Also fixes NEC-177: Sign-in code box is squeezed to a sliver by the Sign in button
Also fixes NEC-178: Skipped-attachment placeholder is removed as soon as it is drawn
Also fixes NEC-179: Share link row shows only Copy when an owner opens the organisation page
Also fixes NEC-180: Shard CORS refuses PATCH, so share link on/off, role changes and folder names fail from the plugin
Also fixes NEC-181: A plugin refused as too old shows Connected, because the router drops update-required
Also fixes NEC-182: Status icon says Connected, not Device limit reached, when an organisation refuses this device

Changelog

The user guides now show a screenshot for nearly every step, and six plugin and server problems they uncovered are fixed.

Task: NEC-170 — https://projectron.nerchure.com/tasks/178 Every user guide now shows the thing each step asks the reader to find: 58 new screenshots and 18 retaken, all from one reproducible journey in packages/e2e/docs-shots.mts through two real vaults, most cropped to a single element. What cannot be reached for real is marked STAGED in the script and in each guide's brief. scripts/guide-labels.test.mjs now requires every guide image to come from docs-shots and none to be orphaned. Capturing them for real found six defects in shipped code, each fixed here with the user's approval, each with its own card, commit and test (mutation-checked both ways): - the sign-in code box was a sliver (NEC-177); - the skipped-attachment Download on this device button was removed as soon as it was drawn (NEC-178); - an owner never saw Replace… or Turn off… on the share link (NEC-179); - the shard's CORS refused PATCH, so share link on/off, role changes and folder names failed from the plugin (NEC-180); - a plugin refused as too old showed Connected (NEC-181); - a device refused by one organisation showed Connected instead of Device limit reached (NEC-182). Spec: `docs/changes/NEC-170-guide-screenshots/spec.md` Also fixes NEC-177: Sign-in code box is squeezed to a sliver by the Sign in button Also fixes NEC-178: Skipped-attachment placeholder is removed as soon as it is drawn Also fixes NEC-179: Share link row shows only Copy when an owner opens the organisation page Also fixes NEC-180: Shard CORS refuses PATCH, so share link on/off, role changes and folder names fail from the plugin Also fixes NEC-181: A plugin refused as too old shows Connected, because the router drops update-required Also fixes NEC-182: Status icon says Connected, not Device limit reached, when an organisation refuses this device ## Changelog The user guides now show a screenshot for nearly every step, and six plugin and server problems they uncovered are fixed.
Task: NEC-170

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
The Sign in button is full width everywhere else, and in the code row it
took all of it, leaving the box a sliver nobody could see a code in.

Task: NEC-170
Also-fixes: NEC-177

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
Task: NEC-170

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
Drawn inside the embed, the Download on this device button was replaced by
Obsidian's own "could not be found" as soon as the embed loaded, so a file
skipped on this device looked lost from inside the note.

Task: NEC-170
Also-fixes: NEC-178

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
Started while the page was still detached, the load was dropped as stale,
so an owner saw Copy alone and never Replace or Turn off.

Task: NEC-170
Also-fixes: NEC-179

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
The plugin turns the share link off and on, changes roles and publishes
folder names with PATCH, and each failed its preflight in Obsidian with
"Failed to fetch" before reaching the server.

Task: NEC-170
Also-fixes: NEC-180

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
The router ranked every state but update-required, so a plugin the server
refused as too old fell through to connected and showed green while
nothing synced.

Task: NEC-170
Also-fixes: NEC-181

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
A refused organisation's connection is dropped, so the provider's
aggregate was of the others, and the icon said Connected while that
organisation's folders synced nothing.

Task: NEC-170
Also-fixes: NEC-182

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
Every image comes from one reproducible journey through two real vaults,
cropped to what a step asks the reader to recognise. What cannot be
reached for real is marked STAGED in the script and in the guide's brief.

Task: NEC-170

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
Task: NEC-170

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
A membership refresh collapsed every state it did not name into
disconnected, so one finishing after the server refused this build replaced
update required. The skipped-attachment button, now that it stays on screen,
removes itself when the download works and offers again when it does not.

Task: NEC-170
Also-fixes: NEC-181
Also-fixes: NEC-178

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
Say which guide shots rest on a staged plan, and name the organisation again
Some checks failed
CI / promote (pull_request) Has been cancelled
CI / build (pull_request) Has been cancelled
CI / e2e (pull_request) Has been cancelled
Release note / release-note (pull_request) Successful in 14s
9c28cc5b2c
Task: NEC-170

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
Release notes for #236: the guides' screenshots and the six fixes they found
Some checks failed
Release note / release-note (pull_request) Successful in 13s
CI / build (pull_request) Successful in 4m58s
CI / e2e (pull_request) Failing after 6m54s
CI / promote (pull_request) Has been skipped
fe40406100
Task: NEC-170

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
Skip the code box check on the previous release, which has the sliver
All checks were successful
Release note / release-note (pull_request) Successful in 13s
CI / build (pull_request) Successful in 4m59s
CI / e2e (pull_request) Successful in 6m54s
CI / promote (pull_request) Has been skipped
Deploy site / deploy (push) Successful in 53s
CI / build (push) Successful in 5m21s
CI / e2e (push) Successful in 6m55s
CI / promote (push) Successful in 31s
9508f78ba0
Task: NEC-170
Also-fixes: NEC-177

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vwesg6bGRVdRhSqFb768ug
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
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
Nectenda/nectenda!236
No description provided.