fix(assets): field-level validation on the New Asset form #222

Merged

nrupard merged 1 commit from fix/mapps-216-new-asset-form-field-validation into main

2026-06-16 16:44:04 +02:00

David commented

2026-06-16 16:16:29 +02:00

Owner

Replace the single shared error banner used for client validation on the New Asset form with per-field error state. Empty required Type and Company are now flagged on their own fields (highlighted, message under the field) instead of as a generic top banner; the banner is reserved for server/submit-time errors. All fields are validated and their errors collected before returning, so the form no longer aborts on the first failure.

Name, Serial Number, Manufacturer, and Model gain maxlength (and matching inline length checks) via documented caps assumed to mirror the server's asset columns. Script-y Serial/Name already render inert on the asset detail page because Dioxus escapes text nodes by default.

Add unit tests covering the new asset validators.

#MAPPS-216

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

Replace the single shared `error` banner used for client validation on the New Asset form with per-field error state. Empty required Type and Company are now flagged on their own fields (highlighted, message under the field) instead of as a generic top banner; the banner is reserved for server/submit-time errors. All fields are validated and their errors collected before returning, so the form no longer aborts on the first failure. Name, Serial Number, Manufacturer, and Model gain `maxlength` (and matching inline length checks) via documented caps assumed to mirror the server's asset columns. Script-y Serial/Name already render inert on the asset detail page because Dioxus escapes text nodes by default. Add unit tests covering the new asset validators. #MAPPS-216 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

David added 1 commit

2026-06-16 16:16:29 +02:00

fix(assets): field-level validation on the New Asset form

Check / clippy + fmt + tests (pull_request) Successful in 1m8s

Details

Create release / Create release from merged PR (pull_request) Has been skipped

Details

0ea7c78c67

Replace the single shared `error` banner used for client validation on the New Asset form with per-field error state. Empty required Type and Company are now flagged on their own fields (highlighted, message under the field) instead of as a generic top banner; the banner is reserved for server/submit-time errors. All fields are validated and their errors collected before returning, so the form no longer aborts on the first failure.

Name, Serial Number, Manufacturer, and Model gain `maxlength` (and matching inline length checks) via documented caps assumed to mirror the server's asset columns. Script-y Serial/Name already render inert on the asset detail page because Dioxus escapes text nodes by default.

Add unit tests covering the new asset validators.

#MAPPS-216

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>