fix: annotation layer modal err handling

[riahk]

SUMMARY

• Handles errors on save/update so modal does not close when an error is encountered
• utilizes useEffect hook for initialization to reduce API requests

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

Manual Test:

1. Create or update annotation/annotation layer with duplicate name
2. Attempt to save
3. Confirm that an error toast is thrown and the modal does not close

ADDITIONAL INFORMATION

• Has associated issue:
• Changes UI
• Requires DB Migration.
• Confirm DB Migration upgrade and downgrade tested.
• Introduces new feature or API
• Removes existing feature or API

[codecov-io]

Codecov Report

Merging #12341 (f024403) into master (08183df) will increase coverage by 0.75%.
The diff coverage is 55.88%.

@@            Coverage Diff             @@
##           master   #12341      +/-   ##
==========================================
+ Coverage   72.13%   72.88%   +0.75%     
==========================================
  Files         898      597     -301     
  Lines       45681    21286   -24395     
  Branches     5495     5501       +6     
==========================================
- Hits        32952    15515   -17437     
+ Misses      12517     5644    -6873     
+ Partials      212      127      -85     

Flag	Coverage Δ
cypress	57.64% <100.00%> (?)
javascript	62.72% <54.41%> (+0.14%)	⬆️
mysql	?
postgres	?
python	?
sqlite	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...rset-frontend/src/components/ButtonGroup/index.tsx	100.00% <ø> (ø)
...et-frontend/src/components/ErrorBoundary/index.jsx	95.45% <ø> (ø)
...-frontend/src/components/Select/Select.stories.tsx	0.00% <0.00%> (ø)
superset-frontend/src/components/Select/Select.tsx	90.81% <ø> (ø)
superset-frontend/src/components/Select/styles.tsx	86.30% <ø> (+24.65%)	⬆️
...et-frontend/src/components/Timer/Timer.stories.tsx	0.00% <0.00%> (ø)
superset-frontend/src/components/Timer/index.tsx	95.83% <ø> (ø)
...ews/CRUD/annotationlayers/AnnotationLayersList.tsx	77.35% <33.33%> (-0.53%)	⬇️
...tend/src/views/CRUD/annotation/AnnotationModal.tsx	63.23% <61.53%> (-3.44%)	⬇️
...ews/CRUD/annotationlayers/AnnotationLayerModal.tsx	69.79% <64.00%> (-5.79%)	⬇️
... and 519 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 391d909...dec3e6c. Read the comment docs.

[ktmud]

Maybe we can split these if conditions into multiple early returns. Currently it's kind of hard to reason with.

[eschutho]

you can also use optional chaining to help if you want.