Nice, @dgmapping!

The problem which I've fixed would cause that. Wait a bit until routing service updates it's graph.

Hi, currently this road is marked as motor_vehicle=destination which means that it's not accessible to motor vehicles unless they intend to park somewhere there.

I think it was intended for a small part near the school. I'll remove it for the most of the street.

Estonia has no motorways (kiirtee) where specific traffic law rules apply (no cycling, no walking, no driving under 40km/h, no stopping, etc.). We do have traffic rules for them if some day we'll get one.

Routers may apply such defaults to this road which would be incorrect.

I vote for reverting this change.

Hi and welcome!
You've made a hole in the building at shop's location.

Shops don't require multipolygons with a buliding and you shouldn't change the building in any way if you want to add shop separately. It's also better to map such shop as just a node.

I've fixed it, you can check it out.

access=private

may be also appropriate if there is a gate.

Hi, you need to add a highway**=* tag to actually turn this line into a road.
As I understand it's behind the fence so

highway=service
service=driveway

would be appropriate.

You can either add it in Tags section when you select the line or by clicking on it under "Feature type" and searching for "driveway"

also consider using landuse=grass instead of a grassland, if the land is cared for (mown)

Hi,
way/1338952157
this grassland shouldn't have a name.

https://www.mapillary.com/app/?lat=58.05724330000001&lng=25.36766130000001&z=17&pKey=6600907689965680&focus=photo

Hi, I can't see any signs which would make the road private. If there are no signs you can't really claim that or stop someone from making a u-turn there.

Hi, addresses can exist without a building. Please don't remove them without checking with Maa-amet

https://xgis.maaamet.ee/adsavalik/CU00445385

Hi, it looks like the shop is duplicated. Is there a reason why you've added two nodes?

I'd recommend going to tags (at the bottom when you select a feature),
1) restoring
amenity=community_centre
community_centre=cultural_centre
or maybe
community_centre=language_centre

community_centre=*

2) change
school:type=keeltekool
to
training=language

Unfortunately, most map applications categorize feature as a single category, so this will likely be a cultural center or a shop depending on the application.

One way to overcome this could be adding the shop separately, as a new node inside the building and removing shop tags from the cultural center.

I don't think this would be a problem since gas stations often have both gas station feature and a convenience shop feature.

Hi, is Hispaania maja not a community centre anymore? I'm confused that you've removed

amenity=community_centre
community_centre=cultural_centre

Oh, sorry. Didn't notice the second changeset. All is correct! Thank you.

Hi and welcome. disused:shop=* should be used instead of the shop tag.

like this
disused:shop=bakery
name=Blondi Bakery

leaving shop=bakery will make most data consumers (apps) think that the shop still operates.

It's up to the search to support Estonian language and name it correctly, but maybe you can cheat the system, add Hageri in front and say that everyone calls it that. I wouldn't be a big fan of such solution, but it's also hard to say something against it.

Hi, about climbing adventure,
way/1332867876

The name looks like a classification name of what should be in the tags. If you really need to emphasize that it's a low climbing adventure without safety harness, we could invent a new tag.

And maybe it's better to tag it as a pitch, not a sports_centre

Exactly as you've described. Those require non-default conditions and have a default maxspeed sign along with a non-default, conditional one.