Edit manufacturers #73

[MatteoGuarnaccia5]

Description

Edit a manufacturer logic. Dialog opens when edit icon clicked, which is prepopulated. Users can edit fields and sends patch request. Error handling in place to ensure name cannot be edited to one that already exists in database, that at least one change is made, and that required fields are not left blank

Testing instructions

Add a set up instructions describing how the reviewer should test the code

• Review code
• Check Actions build
• Review changes to test coverage

Agile board tracking

closes #73

[codecov]

Codecov Report

Attention: 114 lines in your changes are missing coverage. Please review.

Comparison is base (9688e29) 89.33% compared to head (d97f0b5) 85.10%.
Report is 30 commits behind head on develop.

❗ Current head d97f0b5 differs from pull request most recent head 941052f. Consider uploading reports for the commit 941052f to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #112      +/-   ##
===========================================
- Coverage    89.33%   85.10%   -4.23%     
===========================================
  Files           32       34       +2     
  Lines         1519     1853     +334     
  Branches       461      534      +73     
===========================================
+ Hits          1357     1577     +220     
- Misses         159      273     +114     
  Partials         3        3              

Files	Coverage Δ
src/app.types.tsx	100.00% <ø> (ø)
...anufacturer/deleteManufacturerDialog.component.tsx	100.00% <100.00%> (ø)
src/manufacturer/manufacturer.component.tsx	93.33% <92.00%> (-6.67%)	⬇️
src/mocks/handlers.ts	95.86% <89.28%> (-1.60%)	⬇️
src/api/manufacturer.tsx	82.27% <83.07%> (+3.70%)	⬆️
src/manufacturer/manufacturerDialog.component.tsx	50.50% <50.50%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[joelvdavies]

Worked well for me, just some comments on the code and unit test coverage. A few things I will leave for edit to avoid more conflicts.