Skip to content

Caption remaining diagrams in two cash-register-integration pages - #86

Open
deboragracio wants to merge 2 commits into
mainfrom
docs-captions-cri-fix
Open

Caption remaining diagrams in two cash-register-integration pages#86
deboragracio wants to merge 2 commits into
mainfrom
docs-captions-cri-fix

Conversation

@deboragracio

Copy link
Copy Markdown
Collaborator

What

Follow-up fix to the captioning PRs (#81 etc.). Two pages carried a single pre-existing (legacy) caption, so during the captioning pass they were treated as "already captioned" and their other flow diagrams were left without captions. This completes them.

File Before After
general/cash-register-integration/cash-register-integration-regular-workflow.md 1 figure 8 figures
middleware-de-kassensichv/cash-register-integration/single-receipt-creation.md 1 figure 10 figures
  • Every diagram now has a *Figure N.* caption, numbered per file in document order (the previously-existing caption is renumbered into sequence).
  • Descriptions derived from each diagram's section heading / filename.
  • No other content changed.

Why it slipped through

The caption tooling skipped files that already had ≥1 caption, and the lint checked table coverage but not figure coverage. Caught during review of the rendered regular-workflow page.

🤖 Generated with Claude Code

Follow-up to the captioning PRs: these two pages had a single legacy
caption, so their other flow diagrams were left uncaptioned on main.
Caption every diagram, numbered per file in document order:
- cash-register-integration-regular-workflow.md: Figures 1-8
- single-receipt-creation.md: Figures 1-10

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@deboragracio
deboragracio requested a review from a team as a code owner August 12, 2026 14:04
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