Repository navigation
docs(blueprints): refresh S04/S07/S08 wireframes, complete the inventory and add a blueprint check - #167
Merged
Conversation
…ory and add a blueprint check Refs #23. - S04, S07 and S08 wireframes no longer matched the app: they predated the hire relay (#144), wallet sign-in (#149), the funds check (#159) and the Studio builder (#145). They are regenerated from the app at 1440 and 390 px; the other 30 generated wireframes still match and are unchanged. - screens.md: S04 states now cover the wallet sign-in step, the funds check before any signature, resuming an unfinished hire, and the confirming/held result. S06 lists HireDetailView. The inventory adds HireDetailView and Puls3Mark and explains the (new) planned components. - The wireframe tool fills a valid manifest before opening the deploy sheet, and seeds the demo wallet so the S08 PNG is the same on every run. - scripts/tests/blueprints.test.sh checks offline that every screen ID has a 1440 and a 390 px PNG, every wireframe is linked and every link resolves, every screen documents its four states, data, actions and components, and every Entity.field exists in docs/domain/model.md.
6 tasks
TOMOKI977
approved these changes
Oct 11, 2026
TOMOKI977
left a comment
Contributor
There was a problem hiding this comment.
Approved. The reviewed scope is complete, the current head is mergeable, and all applicable checks are green.
Deploying puls3 with
|
| Latest commit: |
06f44be
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://ce006db7.puls3-4lw.pages.dev |
| Branch Preview URL: | https://docs-23-screen-specs-complet.puls3-4lw.pages.dev |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #23
What
HireDetailView. The inventory addsHireDetailViewandPuls3Markand explains the (new) planned components (S11, S12).scripts/tests/blueprints.test.sh(CIscriptsjob) verifies that every screen ID has a 1440 and a 390 px PNG, every wireframe is linked and resolves, every screen documents its four states, data, actions and components, and everyEntity.fieldexists indocs/domain/model.md.Acceptance criteria (#23)
docs/domain/model.md(checked in CI)docs/blueprints/wireframes/are the accepted replacement: they are rendered from the app's real components with one command and checked in CI, so they stay in sync with the codeVerification
bash scripts/tests/blueprints.test.sh: 262 passed, 0 failed (a removed PNG is caught)flutter test tool/wireframes/wireframes_test.dart: every wireframe matches the app, on two consecutive runsFollow-up (app, not docs)
On phones the bottom navigation bar covers the end of the deploy sheet (see
S08-deploy-sheet-390.png): the same fix as the hire sheet in #144.