A lot of this residential area isn't, it needs breaking into managable areas and surveying properly.

I am not sure I would call The Golden Mile residential.

Probably just a normal bridge, there would be no need for a drawbridge for canal traffic.

Tags such as access=yes, bicycle=yes, cycleway=no, foot=yes, horse=yes, motor_vehicle=yes,
oneway=no are totally unescessary on highway=unclassified as they are general rules.
They only serve to clutter the tagging and make peer review more difficult.

Hi, welcome to OSM.
The names Haulage Yard and Lagoon sound descriptive rather than names.

Hi, welcome to osm
This should be tagged as landuse=orchard, iD does lack presets but you can use your own.

House names probably belong in the addr:housename tag rather than name

Mace Express should probably be shop=convienience

You seem to have added a lot of meadows, it seems unlikely that there are that many in that part of Shropshire. It is more likely to be pastureland used for grazing. Meadow is very special and very rare.

Hi, a small thing. You do need to connect paths to the roads. As you have mapped a cyclist on the cycleway cannot reach New Road.

Hi, welcome to OSM.
Thank you for adding the paths and cycleways, that is very useful. One small thing, you do need to connect them to other ways so that they are useful to routers.
Thanks Phil

Hi Eric. I think Brooklside Mobile Home Park should be tagged as residential=caravan_site.
I don't think its a place where people go on holiday.
Cheers Phil

The supermarket should be tagged as
cuisine=chinese/cantonese, you decide.
Brunch'n'lunch should be amenity=fast_food rather than shop. It also should have a cuisine tag adding.

This should probably have access=private added.

Hi, welcome to OSM.
Thank you for adding the paths, that is very useful. One small thing, you do need to connect them to other ways so that they are useful to routers.
Thanks Phil (former Markfield Boy)

Hi, this changeset had gone badly wrong so I have reverted it and made the simple change that was needed.
You had deleted the roundabouts, in so doing you had lost information such as lane and ref=.
Roundabouts should be tagged as junction=roundabout and not oneway=yes, this allows routers to give correct information. Roundabouts are not tagged as *_link but continue the road classification.
It is also impolite to unnecessarily delete another mappers work to redraw it, it also losses history.

Hi, welcome to OSM, I think something has gone wrong here and suspect the node is misplaced. Bing imagery shows the road ajoining the water and no space for buildings that side of the road.

Hi, welcome to OSM. I saw your note and was about to add this subject to bing imagery looking right, it does.
It seems unlikely that this a a bar, iD is very american and does seem to push bar. For this I would suggest you change to amenity=pub, name="Malt Shovel"
Cheers, keep on mapping.

Hi
Are you sure these roads are only linked by a footway.
Looking at bing imagery I would expect a cycleway as a minimum. Possibly leaving the road in place and adding barrier=bollard that would restrict motor vehicles is closer to what is there.

Hi, welcome to OSM.
You seem to have created a crossing way here. If the cycleway goes under the road, you either need to create a tunnel or bridge, thats your judgement call.