Delete manufacturer #72

[MatteoGuarnaccia5]

Description

Enables users to delete manufacturers. Manufacturers that are a part of a Catalogue Item cannot be deleted and if the user tries to an appropriate error message will be shown

Testing instructions

• Review code
• Check Actions build
• Review changes to test coverage
• Check that deleting a manufacturer that is part of an catalogue item is handled

Agile board tracking

closes #72

[codecov]

Codecov Report

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

Comparison is base (9688e29) 89.33% compared to head (8d9b3ab) 88.91%.
Report is 18 commits behind head on develop.

❗ Current head 8d9b3ab differs from pull request most recent head de6c201. Consider uploading reports for the commit de6c201 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop      #97      +/-   ##
===========================================
- Coverage    89.33%   88.91%   -0.43%     
===========================================
  Files           32       32              
  Lines         1519     1524       +5     
  Branches       461      443      -18     
===========================================
- Hits          1357     1355       -2     
- Misses         159      166       +7     
  Partials         3        3              

Files	Coverage Δ
src/app.types.tsx	100.00% <ø> (ø)
...c/manufacturer/addManufacturerDialog.component.tsx	100.00% <100.00%> (ø)
...anufacturer/deleteManufacturerDialog.component.tsx	100.00% <100.00%> (ø)
src/api/manufacturer.tsx	88.88% <93.54%> (+10.31%)	⬆️
src/mocks/handlers.ts	94.44% <86.36%> (-3.02%)	⬇️
src/manufacturer/manufacturer.component.tsx	75.00% <66.66%> (-25.00%)	⬇️

... and 11 files with indirect coverage changes

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

[joelvdavies]

A couple of minor things, and some missing coverage. There are also some conflicts, and add is broken but I suspect that would be fixed by merging develop in. Edit will also need this merged into it afterwards.