feat(#999): read-only Data-eigenaar on inquiry edit#261
Merged
Conversation
Surface the owning organization (attribution.dataOwnerName, added in #973) as a read-only field under Uitvoerder on the inquiry edit screen. The API already returns it; this was purely a UI gap. Kept out of formData so the strict validation schema and the submit body are untouched — the data-owner is assigned server-side, not edited here. Shown only when editing (a new inquiry has no owner yet); falls back to an "Onbekend" placeholder for rows predating the #973 backfill. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.
What
Adds a read-only Data-eigenaar field to the inquiry edit screen (Step 1), under the Uitvoerder select — the second half of Laixer/FunderMaps#999.
The owning organization (
attribution.dataOwnerName, introduced by #973) was already returned byGET /api/inquiry/:idbut never surfaced in the UI. This was purely a frontend gap.Notes
formDataso the.strict()validation schema and the submit body stay untouched — the data-owner is assigned server-side, not edited here.v-if="!isNew") — a brand-new inquiry has no owner yet.Onbekendplaceholder for rows predating the #973 backfill.Issue #999 status
Merging this completes the issue.
Verification
pnpm build(vue-tsc typecheck + vite) passes green. No test runner in this repo; the live edit form was not driven in a browser (auth-gated, needs a running API).Closes Laixer/FunderMaps#999
🤖 Generated with Claude Code