feat(support): onboardinglijst gelijk aan de app + wizardstatus - #53
Merged
Conversation
De app kent negen school-/instructeuritems; het supportportaal toonde er zes. Lesduur, planningsmodus en facturatie ontbraken. Daardoor kon het supportscherm "klaar" suggereren terwijl er in de app nog stappen openstonden. Dat is geen cosmetisch verschil. Rijschool Nielsen stelde op 30 juli een lesduur van 90 minuten in, een week na inschrijving. In het portaal was die school "ingeschreven, ingelogd, verder niets" — en de conclusie "ze zijn nooit begonnen" was dus verkeerd. Ze kwamen terug, stelden iets in, en stopten daarna. Dat is een ander gesprek met die klant. GEREED LOS VAN WANNEER. Elke stap heeft nu een expliciete `gereed`. Bij de lesduur is "gedaan" af te leiden (het veld is gevuld) maar het tijdstip niet exact: de planningsrij kan bij een andere instelling zijn ontstaan. Liever een stap die eerlijk "gedaan" zegt zonder datum dan een verzonnen tijdstip. Voorheen leidde het scherm "gedaan" af uit `wanneer is not null`, en dan moet je wel iets verzinnen. Gemeten dat de drie rijen pas ontstaan als er echt iets wordt ingesteld: Liamdrive (deed niets) heeft geen van de drie; Zwaantje kreeg ze op 9 minuten, 1 uur en 12 uur na inschrijving. created_at is hier dus een betekenisvol moment en geen bijproduct van de schoolcreatie. Verder: de setupwizard-status (afgerond of overgeslagen, met tijdstip) staat nu in het schoolblok. Sinds de OTA van 5 aug is dat het antwoord op "heeft deze rijschool de setup gezien, en wat deed hij ermee". Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe support school detail function now returns explicit onboarding completion flags, optional timestamps, and setup-wizard status. The support page uses these fields to detect stalled steps and render onboarding and wizard states. ChangesSupport school detail
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant SupportPage
participant support_school_detail
participant SchoolData
SupportPage->>support_school_detail: Request school detail
support_school_detail->>SchoolData: Read onboarding and wizard data
SchoolData-->>support_school_detail: Return source records
support_school_detail-->>SupportPage: Return JSONB detail
SupportPage->>SupportPage: Render completion and wizard states
Possibly related PRs
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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
1 participant
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.
Sluit het gat dat ik gisteren zelf meldde: de app kent negen school-/instructeuritems, het supportportaal toonde er zes.
Wat erbij komt
Drie ontbrekende stappen — lesduur, planningsmodus en facturatie — met dezelfde afleiding als
src/lib/onboardingStatus.tsin de app. En de setupwizard-status in het schoolblok: afgerond, overgeslagen of nog niet doorlopen, met tijdstip.Waarom dit geen cosmetica is
Rijschool Nielsen stelde op 30 juli een lesduur van 90 minuten in, een week na inschrijving. In het portaal was die school "ingeschreven, ingelogd, verder niets". De conclusie die ik daaruit trok — ze zijn nooit begonnen — was verkeerd. Ze kwamen terug, stelden iets in, en stopten daarna. Dat is een ander gesprek met die klant.
Voorproef tegen productie:
gereedstaat nu los vanwanneerBij de lesduur is "gedaan" af te leiden (het veld is gevuld) maar het tijdstip niet exact — de planningsrij kan bij een andere instelling zijn ontstaan. Liever een stap die eerlijk "gedaan" zegt zonder datum dan een verzonnen tijdstip.
Voorheen leidde het scherm "gedaan" af uit
wanneer is not null; dan moet je wel iets verzinnen. Nu draagt elke stap een explicietegereed, en toont het scherm "gedaan" wanneer er geen betrouwbaar tijdstip is.Gemeten dat
created_athier wél betekenis heeft: de drie rijen ontstaan pas als er echt iets wordt ingesteld. Liamdrive (deed niets) heeft geen van de drie; Het Zwaantje kreeg ze op 9 minuten, 1 uur en 12 uur na inschrijving. Geen bijproduct van de schoolcreatie dus.Bewijs
227 tests groen,
tscschoon, eslint schoon, build groen. De drie nieuwe afleidingen zijn read-only tegen productie gedraaid (tabel hierboven).Migratie
20260805010000— alleensupport_school_detailvervangen (create or replace, zelfde signatuur). Geen tabelwijziging.🤖 Generated with Claude Code
Summary by CodeRabbit