add four new appreciation options to the dialog - #910
Conversation
|
Note on order: this needs BE 845 to merge first. Until then, the four new options appear in the dialog, but saving one returns a 400, since the request validation schema still lists only the original three values. |
ReviewSmall, clean addition — four new Good news on the blocker you noted: be#845 ("Unblocks #910") merged on 2026-08-07 and adds the four new values to the backend's request validation schema — be#837 is now closed. One thing worth checking before merging/testing this: as of right now the deployed |
…appreciation-options
|
Checked the deploy. Dev has it; dev.need4deed.org/swagger/json lists all four values, so this works end to end there. Production does not; main has not moved since 22 July and is now 109 commits behind develop. |
Description
Adds the four new options to the Add Appreciation dialog: Need4Deed certificate, cap, notebook and city certificate. Previously, only tote bag, t-shirt, and benefit card were offered.
Blocked on the backend.
Saving one of the new options currently returns a 400.
sdk-types.jsonin the backend still lists only the old three values, so the request is rejected during validation. Issue: 837. Everything else is in place; the database migration and the SDK both have all seven.Related Issues
Closes #872
Changes
APPRECIATION_TYPESinAppreciationDialog.tsxcityCertificatecorrected to SenatsurkundeScreenshots / Demos
Checklist