Hi, welcome to OSM - noticed you had requested a review.
It seems like you have drawn a road on top of another one, and not added any tags to say what it is.
Perhaps you'd like to delete one of them and add highway=service tags to the other?

Hi, not sure if you're aware but you've added a duplicate Starbucks - just one is needed, either on the building or node.

Agreed, saw this before and thought it was a bit strange

Not sure what the point in the change was, you just seem to have deleted a single node?

Saw a picture on twitter with the bollards preventing car access so I've changed this to a cycleway

Hi, not sure if you're aware but in the normal iD editor on osm.org there is updated DG aerial imagery - which I've used to trace the running track you added.

Why did you delete way/89699559/history ?

Hi, can I ask what is the reason for this change?

Hi, this section is already part of the John Muir Way relation and therefore does not need a name relation/49215#map=13/55.9970/-2.6103

Hi welcome to OSM and thanks for the edits. I see you've tagged way/526840457 as an "unknown road" - looks like it should be highway=track instead?

Hi, just in case you're unaware the tag for an asl seems to be cycleway=asl node/5088895560

Hi, not sure if you are aware but this area has some newer DG Imagery if you want to add the proper building and car park area.

Hi, you were correct the first time with landuse=allotments on way/524382717/ - I've fixed it for you :)

Hi, just to let you know that way/495728700 should be landuse=allotments :)

Hi, just to let you know the correct tag for way/386788476 is landuse=allotments :)

Why have you added a path which goes through a building and doesn't connect to the street which is crosses?

Hi, regarding way/525460410 - I don't think you need to split the road around the crossing island

Is this really a meadow? way/524773837 Looks like farmland to me...

Ist das wirklich eine wiese way/524773837 Sieht aus wie Ackerland zu mir ...

Hi, thanks for the edit and welcome to OSM. I've moved the "Core Path" name to the designation tag, as I assume that's not the actual name of the path?

Hi, just wondering why you have added maxheight tags without a specific value and used "(below_)default" instead?