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.

thanks, removed

There are nothing on this way/124611773#map=17/59.477632/24.838685 way making it a cycleway. Will change to footway. I'll put bicycle=yes, since it's probably safer than the path closer to the beach where many people walk.

If a client cannot consume correct values, it's a problem with the client, we should not cater to a specific usecase and IMO it's better to use, so to speak, law-safe values. Also, in your use case you still need to change the speed in the winter or you are breaking the law. Speed cameras' locations are irrelevant.

I don't think it's a good idea to add addresses to entrances

Let's, but maybe do it the other way around. Clients not supporting conditional speed shouldn't display higher speed than it is.

way/1329453843

no bicycle=designated tag

It seems you've accidentally changed the direction of this way which made oneway work in incorrect direction
way/1328878990

Also it should be possible to go on bicycle in direction I've indicated with red arrow, but those small two blue places won't allow routing due to them being one-way.

https://drive.google.com/file/d/1Q2A6dQMYOq1Nzyy8Gm1qWTTl4zvf0Isx/view?usp=sharing

https://scontent-hel3-1.xx.fbcdn.net/v/t39.30808-6/464789610_1075362797628421_7832533533403451347_n.jpg?_nc_cat=102&ccb=1-7&_nc_sid=bd9a62&_nc_ohc=xQ6eR15suSAQ7kNvgHZp2I-&_nc_zt=23&_nc_ht=scontent-hel3-1.xx&_nc_gid=AIJRZAFKuTqBVBwknZGUoXW&oh=00_AYDKCP3iRh2lyESLMrICBEXbRjOq4hmPNRT3oJrrczWnPw&oe=672D2703

Hi,
This is a narrow cycleway, no pedestrian access
way/1329453833
I usually map such cycleways directly on carraigeway with cycleway=track, don't know why I've mapped this one seprately

Hi,
did the cemetery expand to this ground?

If so it's better to make it one cemetery, not two cemeteries near each other with the same name.

You can achieve this in two ways.
1) Remove the new smaller cemetery and expand the bigger one to this new area.
2) If you really don't want to include road in the cemetery, then you can use multipolygons. It's a bit more complex: osm.wiki/Relation:multipolygon
In this case, you'll have both areas as part of multipolygon relation and they both will have "outer" role.

Hi, when specifying tag values, it's better to use consistent names. Otherwise programs can't process this data in a consistent way.

In this case, I'd suggest changing shop's value to just "charity"

Please consult the wiki in such cases.
shop=*

Hi, what you've changed is the whole Village name. Be careful next time, I'll rollback this right away.