It's probably better to revert to historic=building or use historic=yes

Thanks for checking and updating this and other closed businesses.

As you asked for a review of this changeset, I have a couple of suggestions and have included links to the documentation below. I should stress that nothing you have done is incorrect or urgently needs to be changed - the important thing is that features which no longer exist have been removed from the map.

Rather than removing the amenity=* or shop=* tag, you could use a lifecycle prefix like disused, so in this case it would be disused:amenity=fast_food. This gets picked up by the StreetComplete app, so users will be prompted to check for a new business.

You could also change the name tag to old_name.

Although you have the survey date in the changeset comment, you can also put it in the check_date tag, e.g. check_date=2023-04-28. This is also used by tools like StreetComplete.

It's probably better to add landuse as a separate polygon than to add it as a tag to an existing object ("One feature, one OSM element").

osm.wiki/Lifecycle_prefix
old_name=*
check_date=*
osm.wiki/One_feature,_one_OSM_element

I fail to see the point of importing way/1042953053. a short and unconnected length of footway. Deleted in changeset/135419981
---
#REVIEWED_BAD #OSMCHA
Published using OSMCha: https://osmcha.org/changesets/118822927

Welcome to OpenStreetMap and thanks for adding your business. In order for it to "work", you need to add tags to display an appropriate icon on the map and tell data consumers what you are. In this case, this would be leisure=sauna (link to documentation below).

If you would like any help with this, please feel free to ask.

leisure=sauna

Hi, welcome to OpenStreetMap and thanks for updating Nashes Farm Lane.

The original highway=unclassified tag doesn't imply anything about the usability of the road. It fits on the hierarchy of roads at about the same level as highway=residential. From the definitions in the OSM wiki (links below), this does not look like it should be highway=tertiary.

You could add surface=asphalt to the road, as the surface is currently untagged. There's also smoothness=*, but that's more of a subjective tag.

I hope that's useful. If you would like any help, please don't hesitate to ask.

highway=unclassified
highway=tertiary
surface=asphalt
smoothness=*

Thanks!

I fail to understand the point of adding way/1042946426. Not only was this part of the C3/CS3 cycle track already mapped as way/194387797, the added way was not routeable as it was neither tagged as a highway=*, nor was it connected to any other OSM object.

Deleted in changeset/135379579
---
#REVIEWED_BAD #OSMCHA
Published using OSMCha: https://osmcha.org/changesets/118822080

Hi, welcome to OpenStreetMap and thanks for updating the library.

For shops and amenities which have closed, you can use a lifecycle prefix rather than changing the name, so in this case you could change
amenity=library -> was:amenity=library instead of altering the name tag. That makes it easier for data consumers to understand that this is no longer a library.

osm.wiki/Lifecycle_prefix

If it's of any use to you, you can find less useful nodes imported from TfLCID with this Overpass query:
https://overpass-turbo.eu/s/1u5k

Thanks, I'm afraid that the project to use contractors to conflate TfLCID data with OSM on behalf of TfL resulted in same very low quality edits. Fortunately, they appear to have quietly abandoned it, but there's still a lot to fix.

osm.wiki/TfL_Cycling_Infrastructure_Database

Rather than using opening_hours=closed, there's the option of using a lifestyle prefix for features which are no longer open, or have been removed entirely. StreetComplete will prompt passing users to re-check some of these at intervals.

In this case, you could consider changing shop=toys to disused:shop=toys and possibly add check_date=2023-04-21

osm.wiki/Lifecycle_prefix
check_date=*

Hi, welcome to OpenStreetMap and thanks for adding your business to the map.

I've moved the address and contact details from the description to the appropriate OSM tags and added the tag shop=travel_agency. This helps map users and data consumers find you.

Welcome to OpenStreetMap.

Thanks for spotting that and updating the map. You asked for a review of your edit and it looks fine to me.

Subjectively, it might not be suitable for walking, or fun to walk along. However, it is not prohibited: there are no TSRGD Diagram 625.1 "Pedestrians prohibited" signs.

Please don't add fiction to the map. Fully reverted in changeset/135087390

#DWG
---
#REVIEWED_BAD #OSMCHA
Published using OSMCha: https://osmcha.org/changesets/135082845

Thanks for adding it! It may take a couple of week before routing services start to use it.

No need to apologise!

I just wanted to double check, as some people will filter on vegan only in apps which use that tag. I've restored it to "only" in changeset #134916778. There's a little more information on the diet:vegan tag in the wiki page linked below.

If it's something that's of interest to you, there's also a tagging scheme for zero-waste shops.

diet:vegan=*
zero_waste=*

Hi, welcome to OpenStreetMap and thanks for updating the map.

Could I ask why you have changed Pipoca from diet:vegan=only to diet:vegan=yes? This implies that they also sell non-vegan products.

Hi, welcome to OpenStreetMap and many thanks for updating the house details here.

You asked for a review and it looks fine to me.

The only thing which you may want to change is the addr:housenumber tag. There are several ways to tag a building with multiple house numbers (link below), but as another local mapper has already been using a comma, setting it to "29,31" may be the best choice.

osm.wiki/Addresses#Buildings_with_multiple_house_numbers

Thanks!