OpenStreetMap logo OpenStreetMap

Changeset When Comment
99315730 almost 5 years ago

Hi, you requested a review of this change.
Looks good, thanks for adding this, and it looks like you've put good detail on the gate too.
Just a note, the colour should be all lowercase "black", not "Black". I've been in and fixed that for you here.

99319708 almost 5 years ago

Hi, you requested a review of this change. It all looks good to me

99323535 almost 5 years ago

Hi, you requested a review of this change.
Thanks for adding these paths, and looks like you've mostly added lots of good detail to them too.

Just a note about access. It doesn't have to be set for all modes of transport on all roads and paths as a lot of things are implied.
If general access is set, that is implied for all other types of transport, so only anything different has to be set. For things like paths, access for cars/vehicles doesn't need to be set to no because that's implied by the fact that it's a path.

Your canal side path connected to the woods in a couple of places, so I've disconnected that for you.
Also here: node/5353674577 the path you've drawn stops (well it connects to an abandoned railway). If it in fact connects to the path to the south then you should add that too.
And "Leeds and Liverpool Canal - Woodland" seems like an odd name for a path, if that's what's on signs there then that's fine, otherwise maybe it should be the description.

Also, it looks like your changeset comment got cut off a bit. It doesn't have to be really long, just a brief description of the changes you've made and perhaps why.

99360532 almost 5 years ago

Hi, you requested a review of this, and it all looks good to me.

99362521 almost 5 years ago

Hi, you requested a review of this change.
What you need to do here is add a point on the road where you want to split it, then right click on the point and select split (the scissors icon). Then you can change the properties of each section separately.

99372663 almost 5 years ago

Hi, you requested a review of this change.
Here the name is (and should be) tagged on the outline of the whole school area, not on the buiiding.
This is the outline: way/281765046
This is the building: way/281765048
I've put the name on the outline here (and removed it from the building), but it looks like things like the website might need to be updated too, so do change that if you know what the website is now.

99382691 almost 5 years ago

Hi, you requested a review of this edit.
It mostly looks good to me. The postcode should be capitalised and in opening hours it should just be "Sa", not "Sat, I've fixed those things for you.
Also you've listed a lot of things under cuisine, it's normal to only have one or two things there.
I think "sandwich" covers a lot of what you've put in there (that would be used for Subway for example), see here for common values: cuisine=*

99263438 almost 5 years ago

Ah ok, and is there connection to it from the roads it goes near?

99199135 almost 5 years ago

Great stuff!

99199135 almost 5 years ago

Simply select the house and then add details in the address field.
The key things are house number (the 123 box), and/or name if applicable, the street name and the post code if possible.
The city is optional because that can be implied from other boundary data in OSM.

99262790 almost 5 years ago

That makes sense, fair enough, just double checking

99262906 almost 5 years ago

Great, that sounds alright then

99263534 almost 5 years ago

Looks good.
Not really from this changeset, but you've got this bit of grass
way/907115273
mostly attached to the paths around it.
Opinions differ on whether this is best practice (personally I think that it should not be connected and should instead be accurately mapped to the outline of the grass, where possible).
But best to either have it all connected or all disconnected, rather than a mixture.

99263438 almost 5 years ago

Thanks for adding this.
At the moment it's a dead end (which could well be accurate), but it also goes very close to Daffodil Drive and Redwing Close, but you haven't connected it to either of those. If there is connection then that should be added to aid routing.
Also, it was connected to the residential area, so I've disconnected it for you.

99263318 almost 5 years ago

Looks good

99263298 almost 5 years ago

Mostly good, you connected the path to the road, but had also connected the residential area and grass to the road. I've fixed that for you here.

99263269 almost 5 years ago

Looks good. You mention in the changeset description that these are tarmac. That detail can be added to the path, under surface ("asphalt" is used for tarmac).

99263204 almost 5 years ago

Hi, quite minor, but you have the path connected to the fence here:
node/8370801609
I assume that the fence continues south east for a short distance (right click the point, continue line).
Then add the appropriate feature where the path and fence cross (gate or whatever), see here for options: barrier=*#Access_control_on_highways_.28e.g._for_blocking_a_path_or_road.29

99263067 almost 5 years ago

This edit was good, but shortly afterwards you added another bit of grass that overlaps this. These should be merged (select both, right click, merge).

99262939 almost 5 years ago

Similar to a previous one, you've left the construction=residential tag on this, which should be removed now that it's complete.