change it to motor_vehicle=private if it's for vehicles only

Hi, you've made a service way near the river access=private. This forbids pedestrian and bicycle routing there which I think was not your intention.

Hi and welcome! This is not really a skatepark, it's a pump track

https://en.wikipedia.org/wiki/Pump_track

It's already added. This point can be removed.

It's correct though that you've removed the name. If it a skatepark needed here, we shouldn't add description and types of features into names.

About your other edit with a water tap, it's already added too and with better tagging as drinking water, not just unknown tap with unknown water quality. It's added 8 days ago, so you probably can't see it in OrganicMaps. So your water tap can be removed too.

Hi, thank you for you edit, but please, don't add names to places which have no name. Name is not for a description of what it is. Toilets usually don't have any names :)

Hi,
maybe I'll take a look at some point, need to check the sign.

But if the comment is correct, then the "best practice" is really hard to come up with.

I guess
motor_vehicle=private
and maybe
motor_vehicle:conditional=yes @ delivery

But I don't know about delivery. We don't have such thing as a "service transport" in our law and it may be hard to interpret if it means only street cleaning and maintenance, or cars for businesses there and if it includes things like Bolt Food.

Maybe I'll ask transport department after I take a look at the sign.

But anyway, even if Bolt Food is allowed, then
motor_vehicle:conditional=yes @ delivery
is quite a corner case and may not be recognized by whatever routing you are using.

Also, there might be a bollard from Kopli side, which should be added then, needs checking.

Hi!
You should connect new paths to the footway. This allows users to see that they can actually go there and routing software to plot a route through them.

Adding bicycle=yes and foot=yes is not required, it's implied by default for a path.

Hi, bicycle and horse tags you've added are LEGAL restrictions. If there is no road sign, then consider using
mtb:scale=*

to mark path as hard to traverse on bicycle.

managed to do it in the end. reverted.

Somehow OSM OAuth broke down for me and I won't be available in the next three weeks. Hopefully won't forget about this.

The user edits these things in different countries and clearly doesn't know anything about how locals call it. Just mechanical translation which would only confuse end users. I'll revert this changeset.

Hi,

1) opening hours follows a specific syntax

opening_hours=*

you can validate it here

https://openingh.openstreetmap.de/evaluation_tool/?setLng=en

in this case the correct value is "May-Jun"

2)
description=Torn
leisure=park
name=Jaapani lillevaatlusplatvorm

this polygon is really strange. Viewing platform is probably not a park and it shouldn't duplicate the tower node. I suggest you delete it and leave tower

3)
tower:construction=puit
Most tag values should be in English, unless it's about names or descriptions. This is also more like a material, not a type of construction.

Hi, English description already exists under description:en tag. Removed it from the main description.

I don't really understand why would you change https to http

Hi, please take note of tags already existing on the way.
110 km/h speed limit is already mapped for the summer. In OSM we try not to change things back-and-forth as they change in the real world and instead use conditional restrictions and seasonal tag. Not only it allows not to map reoccurring changes all the time, but also allows offline maps to function.

osm.wiki/Conditional_restrictions

Please revert your change.

Hi and welcome!
Some of the wood areas you're adding don't really have many trees and are quite empty. Please be more diligent.

hi, consider using

ruins=yes
instead of
historic=ruins

when the historical value isn't really big.

Hi, if you are adding speed limits, just keep in mind that winter and summer speed limits may differ. I'd leave the winter one as the default one then add something like:

maxspeed:conditional=120 @ (Apr 1-Oct 31) or whatever the dates usually are.

It's probably especially hard to determine the values for roads with dynamic displays instead of signs.

This one probably also needs
maxspeed:variable=yes
maxspeed:variable=*

Hi, Bvvad

Please note that
1) Names are not for descriptions.
2) There is already a description added.
3) Usually, main name should be in a local language.
4) The node type itself already defines what it is (amenity=drinking_water)
5) Родник is natural water exit point to the surface. This is a tap.

I've removed the name you've added.

You can consult the wiki and beginners guide:
osm.wiki/Beginners%27_guide

Hi, small follow-up on this
note/4308711

Hi, about access of this gate.
node/5910633291

Just a small heads-up. bicycle=yes and foot=yes override access=private. They shouldn't be added if the access is private. Vehicles can't pass here anyway so they make no sense. I'll fix it, but just so you know for the future that there is a general access tag and others override its value for the particular mode of transport.

I've verified that there is a code lock on the gate.