What was the source for the left turn only restriction in relation/11566491/history ? From Mapillary I can't see any indication of such (but also can't see any indication that there is none) https://www.mapillary.com/map/im/hUPzG2Q1Jxzvaq35UKk_FA

the source is listed as Maxar but the imagery here is not high enough resolution to make out changeset/90209436 what is your source for adding this? Is it a roundabout or not?

way/840189281/history should inherit all the applicable attributes like name surface etc from the road segment before you split it. I think there may be other issues like this if you could please fix those too.

way/840589945/history should have the road name that was existing before the split, there are other places this happens to if you could please check those too.

way/842043479/ is better mapped as highway=raceway + sport=karting see
highway=raceway

Hi Stephen, for https://osmlab.github.io/osm-deep-history/#/way/597954684 and https://osmlab.github.io/osm-deep-history/#/way/697517823 given https://www.mapillary.com/map/im/ERbSnkfk_IKtrH1acdXD6w shows that it is signposted as for both bicycles and pedestrians, then bicycle=designated is more correct as according to access=*#List_of_possible_values bicycle=yes just means you have legal access whereas bicycle=designated is a stronger form saying both you have access and it's explicitly signposted as for that particular mode of transport.

hi just a tip that it's good practice to reserve the name tag for proper names, so if the house or building has a signposted name that could go into the name field but otherwise street address etc goes into other tags like you have and not the name field. see also osm.wiki/Good_practice#Don.27t_use_name_tag_to_describe_things

thanks. I just made a few extra minor tweaks to try and place the start point more exactly per https://www.mapillary.com/map/im/F5QTIcNn8UEL3Xpf44_HYA

I just checked your recent changesets, looks good now. Thanks.

Hi Ase83, no problem, if you need a hand with reinstating just shout out. It's much easier to do with JOSM there is both an undelete single object option and revert whole changeset option. Then after it's reinstated you can just update the footway to a private driveway and even add a node where the gate is.

Agreed with nevw here, private residential driveways can be mapped in OSM, so it's not right to delete them if they exist on the ground. You also deleted the footway way/174520032/history could you explain why that was done too please?

hi Stephen,

Since you've requested a review, just a comment that "self" no u-turns where it's from one road back onto itself like you added in
relation/11546114/history
relation/11546115/history
relation/11546118/history
relation/11546116/history
generally are not needed (it's already assumed you can't u-turn like that) and unless it's signposted as such usually these shouldn't be mapped.

hi I updated the bounds per the LPI Basemap it looks like you included an apartment building as part of the school, if you have local knowledge if you could confirm that? I also updated the tags of the buildings.

Hi Stephen, this is good, but a few tips since you asked for a review.
1. looks like you have a few short way segments these could probably be joined together into one way unless you need to use different tags for each section.
2. unless the cycleway is closed, i would leave out access=no, although the bicycle=yes would allow bicycles, access=no could be misinterpreted so it's better to not include that in my opinion.
3. if the pedestrians and cyclists share the same space and don't have their own lanes you can add segregated=no

golf=tee is the common tag for tee's so I've changed the tag.

hi you deleted the water areas here, could you please elaborate on why?

If it's a declared Wildlife Protection Area by the local council you can consider adding boundary=protected area eg way/174172877

If you don't use the preset, and just manually enter the tags one by one then it won't auto-add the building tag, otherwise you'll just need to manually remove the building tag if using the preset in iD.

way/839191190 looks like a building not a tower, so I've changed it and then mapped the satellite dishes as nodes, did you want to take a look and see if that's okay?

way/677568458/history you added layer=-1 but the building is marker as a roof and it's the highest object, I don't see why layer is needed at all here unless you have footways on top of the roof or multiple roofs.