UIQM-730 Create/Edit/Derive MARC record - Retain focus when MARC record validation rules error display. Show validation issues toasts.

[BogdanDenis]

Description

Create/Edit/Derive MARC record - Retain focus when MARC record validation rules error display.
Show validation issues toasts.

Screenshots

Issues

UIQM-730

[github-actions]

Jest Unit Test Results

  1 files  ±0   45 suites   - 1   2m 19s ⏱️ -15s
481 tests +2  481 ✅ +2  0 💤 ±0  0 ❌ ±0 
484 runs  +2  484 ✅ +2  0 💤 ±0  0 ❌ ±0 

Results for commit dd7e1ce. ± Comparison against base commit 9c8f180.

This pull request removes 1 and adds 3 tests. Note that renamed tests count towards both.

Given useFocusFirstFieldWithError when a field has an error should focus that field ‑ Given useFocusFirstFieldWithError when a field has an error should focus that field

Given QuickMarcEditor when saving form with validation errors and deleted fields should show a toast notification about validation error ‑ Given QuickMarcEditor when saving form with validation errors and deleted fields should show a toast notification about validation error
Given QuickMarcEditor when saving form with validation warnings and errors should show a toast notification about validation warning and error ‑ Given QuickMarcEditor when saving form with validation warnings and errors should show a toast notification about validation warning and error
Given QuickMarcEditor when saving form with validation warnings should show a toast notification about validation warning ‑ Given QuickMarcEditor when saving form with validation warnings should show a toast notification about validation warning

♻️ This comment has been updated with latest results.

[zburke]

Code changes LGTM but I'd like to tweak the language in the toasts a little bit. Sorry, I was a copy editor in college and the job still haunts me. 👻

[zburke]

• s/warn errors/warnings/: warnings are notifications that something is not quite right, but validation will still succeed. A record with warnings can be saved.
• s/fail errors/errors/: an error means validation failed. A record with errors cannot be saved. Alternative to "errors": "failures", but I think "errors" is more common.

Maybe use language like, "Resolve issues as needed..." since "issues" can cover both "warnings" and "errors"?

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
96.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud