Skip to content

Corrigeer veldnamen sampleverwerking vervolg (#998)#262

Merged
yorickdewid merged 1 commit into
mainfrom
fix/998-veldnamen-vervolg
Jul 10, 2026
Merged

Corrigeer veldnamen sampleverwerking vervolg (#998)#262
yorickdewid merged 1 commit into
mainfrom
fix/998-veldnamen-vervolg

Conversation

@yorickdewid

Copy link
Copy Markdown
Contributor

Vervolg op de veldnaam-correcties uit #998 (eerste ronde zat in #259). Verwerkt de aanvullende wijzigingen uit de laatste comment op het issue.

Naamswijzigingen (Inquiry/SampleForm.vue)

Veld Oud label Nieuw label
constructionLevel Onderkant fundering (m NAP), alleen bij niet-onderheide fundering Onderkant constructie (opbouw op fundering)
foundationDepth Diepteligging funderingshout Onderkant fundering (m NAP), alleen bij niet-onderheide fundering

Gedrag

  • foundationDepth accepteert nu negatieve waarden (:min 0-999.99).
  • Selects terugzetbaar naar geen selectie (-). De gedeelde Select rende de placeholder-optie hardcoded disabled, waardoor je na een keuze niet meer terug kon naar "-". Toegevoegd: opt-in clearable prop die de placeholder selecteerbaar maakt (zet het model terug op null).
    • Toegepast op alle 23 optionele "-" selects in de inquiry-sampleform — de 5 scheurvelden uit het issue plus de overige optionele velden (onderbouw, paaltype, houtsoort, houtaantasting, kwaliteits-/draagkrachtvelden, …) voor consistente UX.
    • De verplichte Step1-prompts ("Kies een type/uitvoerder/beoordelaar") blijven bewust niet-clearable.

Test

  • pnpm type-check

Closes #998

🤖 Generated with Claude Code

Verwerkt de aanvullende wijzigingen uit issue #998:

- constructionLevel: label -> "Onderkant constructie (opbouw op fundering)"
- foundationDepth: label -> "Onderkant fundering (m NAP), alleen bij
  niet-onderheide fundering"; accepteert nu ook negatieve waarden (min -999.99)
- Select: nieuwe `clearable` prop maakt de placeholder-optie selecteerbaar
  zodat een keuze teruggezet kan worden naar geen selectie (-)
- Alle optionele "-" selects in de inquiry-sampleform zijn `clearable`
  (waaronder de 5 scheurvelden); de verplichte Step1-selects blijven ongewijzigd

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@yorickdewid
yorickdewid merged commit b6e4faa into main Jul 10, 2026
1 check passed
@yorickdewid
yorickdewid deleted the fix/998-veldnamen-vervolg branch July 10, 2026 14:03
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