Skip to content

Bug/am 1432 verander vignet vignette - #766

Open
WouterAms wants to merge 2 commits into
mainfrom
bug/AM-1432-verander-vignet-vignette
Open

WouterAms wants to merge 2 commits into
mainfrom
bug/AM-1432-verander-vignet-vignette

Conversation

@WouterAms

@WouterAms WouterAms commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Changes

Standardizes vignet to vignette in the boat-charging module’s English identifiers and related redirect keys, while preserving Dutch user-facing text.

Changes:

  • Renames types, API hooks, routes, form fields, components, and test identifiers.
  • Replaces old-named screens, action buttons, and alert mappings with renamed equivalents.
  • Updates session initialization, confirmation, and navigation tests to use the new names.
File Description
src/​modules/​redirects/​exports/​RedirectKey.ts Renames vignette redirect keys and values.
src/​modules/​boat-charging/​types.ts Renames vignette types, endpoints, and form fields.
src/​modules/​boat-charging/​service.ts Updates endpoint definitions and exported mutation hooks.
src/​modules/​boat-charging/​screens/​BoatChargingVignette.screen.tsx Updates the management screen and route parameter.
src/​modules/​boat-charging/​screens/​BoatChargingVerifyVignette.screen.tsx Adds the renamed verification screen.
src/​modules/​boat-charging/​screens/​BoatChargingVerifyVignet.screen.tsx Removes the old-named verification screen.
src/​modules/​boat-charging/​screens/​BoatChargingChooseVignette.screen.tsx Adds the renamed selection screen.
src/​modules/​boat-charging/​screens/​BoatChargingChooseVignet.screen.tsx Removes the old-named selection screen.
src/​modules/​boat-charging/​screenConfig.ts Registers renamed screens and header button.
src/​modules/​boat-charging/​routes.ts Renames routes and vignette ID parameter.
src/​modules/​boat-charging/​providers/​BoatChargingVignetteForm.provider.tsx Renames the form provider and number field.
src/​modules/​boat-charging/​hooks/​useInitSessionNextStep.ts Updates field and route references.
src/​modules/​boat-charging/​hooks/​useInitSessionNextStep.test.ts Updates fixtures and navigation expectations.
src/​modules/​boat-charging/​hooks/​useInitSession.ts Reads the renamed session form field.
src/​modules/​boat-charging/​constants/​vignetteInvalidReasonToAlertMap.ts Adds the renamed invalid-reason mapping.
src/​modules/​boat-charging/​constants/​vignetInvalidReasonToAlertMap.ts Removes the old-named mapping.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignettePostalCodeFormInput.tsx Renames the component and test identifier.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteNumberFormInput.tsx Renames the component, field, and test identifier.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteFormVerifyVignetteButton.tsx Adds the renamed verification button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteFormEditVignetteButton.tsx Adds the renamed edit button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteFormDeleteVignetteButton.tsx Adds the renamed delete button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteFormAddVignetteButton.tsx Adds the renamed add button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteFormAddToMyVignettesSwitch.tsx Updates component, type, and test identifier names.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteForm.tsx Connects renamed controls, fields, and redirect key.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetFormVerifyVignetButton.tsx Removes the old-named verification button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetFormEditVignetButton.tsx Removes the old-named edit button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetFormDeleteVignetButton.tsx Removes the old-named delete button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetFormAddVignetButton.tsx Removes the old-named add button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingChooseVignetteFormSubmitButton.tsx Updates selection submission and validation references.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingChooseVignetteForm.tsx Updates selection fields, navigation, and test identifiers.
src/​modules/​boat-charging/​components/​vignettes/​BoatChargingVignetteListItem.tsx Renames props and edit-navigation references.
src/​modules/​boat-charging/​components/​vignettes/​BoatChargingMyVignettes.tsx Uses renamed list items and add route.
src/​modules/​boat-charging/​components/​vignettes/​BoatChargingAddVignetteHeaderButton.tsx Renames the header button and navigation target.
src/​modules/​boat-charging/​components/​session/​BoatChargingSession.tsx Renames the vignette card’s test identifier.
src/​modules/​boat-charging/​components/​BoatChargingHistorySessionDetails.tsx Renames the history card’s test identifier.
src/​modules/​boat-charging/​components/​BoatChargingGuestEmailForm.tsx Updates pending selection and verification routes.
src/​modules/​boat-charging/​components/​BoatChargingConfirmSessionSubmitButton.tsx Updates verification hooks, fields, and alert references.
src/​modules/​boat-charging/​components/​BoatChargingConfirmSessionDetails.tsx Updates vignette details and change-navigation references.
src/​modules/​boat-charging/​alerts.ts Renames alert keys, test identifiers, and redirect reference.

Test instructions

Other notes

@WouterAms
WouterAms requested review from a team and a balanced review from Copilot November 13, 2026 10:49
@WouterAms
WouterAms force-pushed the bug/AM-1432-verander-vignet-vignette branch from a331b63 to c3053e5 Compare November 13, 2026 10:51
@github-actions github-actions Bot added module:boat-charging Module touched based on changed src/modules folder(s). module:redirects Module touched based on changed src/modules folder(s). labels Nov 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

Human confirmation is needed that the backend /links response supplies boatChargingRequestVignette and boatChargingVignetteInfo, since backend evidence is unavailable and missing keys could disable the vignette links.

No open issues 🎉

What changed in this PR

Standardizes vignet to vignette in the boat-charging module’s English identifiers and related redirect keys, while preserving Dutch user-facing text.

Changes:

  • Renames types, API hooks, routes, form fields, components, and test identifiers.
  • Replaces old-named screens, action buttons, and alert mappings with renamed equivalents.
  • Updates session initialization, confirmation, and navigation tests to use the new names.
File Description
src/​modules/​redirects/​exports/​RedirectKey.ts Renames vignette redirect keys and values.
src/​modules/​boat-charging/​types.ts Renames vignette types, endpoints, and form fields.
src/​modules/​boat-charging/​service.ts Updates endpoint definitions and exported mutation hooks.
src/​modules/​boat-charging/​screens/​BoatChargingVignette.screen.tsx Updates the management screen and route parameter.
src/​modules/​boat-charging/​screens/​BoatChargingVerifyVignette.screen.tsx Adds the renamed verification screen.
src/​modules/​boat-charging/​screens/​BoatChargingVerifyVignet.screen.tsx Removes the old-named verification screen.
src/​modules/​boat-charging/​screens/​BoatChargingChooseVignette.screen.tsx Adds the renamed selection screen.
src/​modules/​boat-charging/​screens/​BoatChargingChooseVignet.screen.tsx Removes the old-named selection screen.
src/​modules/​boat-charging/​screenConfig.ts Registers renamed screens and header button.
src/​modules/​boat-charging/​routes.ts Renames routes and vignette ID parameter.
src/​modules/​boat-charging/​providers/​BoatChargingVignetteForm.provider.tsx Renames the form provider and number field.
src/​modules/​boat-charging/​hooks/​useInitSessionNextStep.ts Updates field and route references.
src/​modules/​boat-charging/​hooks/​useInitSessionNextStep.test.ts Updates fixtures and navigation expectations.
src/​modules/​boat-charging/​hooks/​useInitSession.ts Reads the renamed session form field.
src/​modules/​boat-charging/​constants/​vignetteInvalidReasonToAlertMap.ts Adds the renamed invalid-reason mapping.
src/​modules/​boat-charging/​constants/​vignetInvalidReasonToAlertMap.ts Removes the old-named mapping.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignettePostalCodeFormInput.tsx Renames the component and test identifier.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteNumberFormInput.tsx Renames the component, field, and test identifier.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteFormVerifyVignetteButton.tsx Adds the renamed verification button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteFormEditVignetteButton.tsx Adds the renamed edit button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteFormDeleteVignetteButton.tsx Adds the renamed delete button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteFormAddVignetteButton.tsx Adds the renamed add button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteFormAddToMyVignettesSwitch.tsx Updates component, type, and test identifier names.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetteForm.tsx Connects renamed controls, fields, and redirect key.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetFormVerifyVignetButton.tsx Removes the old-named verification button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetFormEditVignetButton.tsx Removes the old-named edit button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetFormDeleteVignetButton.tsx Removes the old-named delete button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingVignetFormAddVignetButton.tsx Removes the old-named add button.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingChooseVignetteFormSubmitButton.tsx Updates selection submission and validation references.
src/​modules/​boat-charging/​components/​vignettes/​form/​BoatChargingChooseVignetteForm.tsx Updates selection fields, navigation, and test identifiers.
src/​modules/​boat-charging/​components/​vignettes/​BoatChargingVignetteListItem.tsx Renames props and edit-navigation references.
src/​modules/​boat-charging/​components/​vignettes/​BoatChargingMyVignettes.tsx Uses renamed list items and add route.
src/​modules/​boat-charging/​components/​vignettes/​BoatChargingAddVignetteHeaderButton.tsx Renames the header button and navigation target.
src/​modules/​boat-charging/​components/​session/​BoatChargingSession.tsx Renames the vignette card’s test identifier.
src/​modules/​boat-charging/​components/​BoatChargingHistorySessionDetails.tsx Renames the history card’s test identifier.
src/​modules/​boat-charging/​components/​BoatChargingGuestEmailForm.tsx Updates pending selection and verification routes.
src/​modules/​boat-charging/​components/​BoatChargingConfirmSessionSubmitButton.tsx Updates verification hooks, fields, and alert references.
src/​modules/​boat-charging/​components/​BoatChargingConfirmSessionDetails.tsx Updates vignette details and change-navigation references.
src/​modules/​boat-charging/​alerts.ts Renames alert keys, test identifiers, and redirect reference.

Review effort: Balanced

Copilot AI balanced review requested due to automatic review settings November 13, 2026 10:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The changed enum string values no longer match the existing /links response keys, causing both vignette links to display an unavailable-function alert instead of opening.

1 open issue

Review effort: Balanced

@github-actions github-actions Bot added the Copilot ready All Copilot comments are resolved label Nov 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Copilot ready All Copilot comments are resolved module:boat-charging Module touched based on changed src/modules folder(s). module:redirects Module touched based on changed src/modules folder(s).

Development

Successfully merging this pull request may close these issues.

2 participants