Add manufacturers #68

[MatteoGuarnaccia5]

Description

Add a manufacturer via a text form, all the fields need to be filled and there is relevant error checks (for invalid urls and duplicate names as well)

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
• test adding and also using invalid url, empty strings(spaces) =

Agile board tracking

closes #68

[codecov]

Codecov Report

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

Comparison is base (9688e29) 89.33% compared to head (d0da1e6) 88.81%.

❗ Current head d0da1e6 differs from pull request most recent head 92ae589. Consider uploading reports for the commit 92ae589 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop      #80      +/-   ##
===========================================
- Coverage    89.33%   88.81%   -0.53%     
===========================================
  Files           32       31       -1     
  Lines         1519     1466      -53     
  Branches       461      433      -28     
===========================================
- Hits          1357     1302      -55     
- Misses         159      161       +2     
  Partials         3        3              

Files	Coverage Δ
src/app.types.tsx	100.00% <ø> (ø)
src/manufacturer/manufacturerDialog.component.tsx	100.00% <100.00%> (ø)
src/api/manufacturer.tsx	87.09% <94.11%> (+8.52%)	⬆️
src/manufacturer/manufacturer.component.tsx	88.88% <83.33%> (-11.12%)	⬇️
src/mocks/handlers.ts	93.96% <75.00%> (-3.50%)	⬇️

... and 11 files with indirect coverage changes

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

[joelvdavies]

Looks good, doesn't work on the current branch of additional-manufacturer-details on the api though as the postCode name needs changing to postcode. May also be worth merging the latest view-manufacturers in as the branch its based on is quite old.

[joelvdavies]

A couple of things I noticed - With the exception of the missing grid elements (and console.log), I am happy for the other changes to be addressed in #112 if you prefer.

[joelvdavies]

Should be merged with #80.