Hi, good edit.
One thing that can be improved is that default name value should be in a local language and it's the English one which should be added separately

Oh, you've removed access=destination from a small part of the street, thanks, didn't understand it at first.

Will also check out Vesivärava today. it's probably motor_vehicle=destination

Hi, could you comment what you've tried to achieve here, I don't understand what exactly mapillary shows here.

Hi, links are used where traffic is separately channelized like on highway ramps and so on, not in the middle of a junction, I'll revert your change.

Thanks to vandalism, I've noticed that
place=city
node/29787492
has inconsistent name:ru now

You may want to cover the whole building with parts. Some consumers ( https://streets.gl/ ) care only about parts and they also don't support the idea of having a default fallback building's tags when parts are missing.

osm.wiki/Simple_3D_Buildings
The entire building outline should be filled with building:part=* areas, tagged with their respective height and other attributes.

I think it's better than nothing, but yes, asphalt is quite curshed there. I don't think that using values in Estonian would be correct though.

I've added dealership tags so that it won't be just a named building

service=dealer
motorcycle:sales=yes
boat:sales=yes

these are based on the website, correct me if I'm wrong.

Moved it a bit based on non-smoking circle around entrance visible on aerials

Hi, grassland is usually in the nature and not tended to.
landuse=grass may be a better choice here, though it's name may be confusing.
landuse=grass
There is also

I don't know if it can be done in StreetComplete, but you can go to node/10943925534, click "Edit", select the crossing and on the left change Feature type from "Point" to "Marked crossing".

Hi, this seems like a recently reconstructed intersection, so I can't really judge from the imagery, but this crossing is probably not informal. Even if it's a crossing where pedestrians don't have a right of way, it's still probably a crossing=uncontrolled

Hi, kind of strange update, only one building is changed in Stockholm while there are other named buildings nearby. And the changeset is gigantic, from Sweden to India.

Hi, please don't put things which don't exist on a map. You are adding them for everyone, not just yourself. I'll revert the change.

building tags are not for ruins. roof kind of can be tagged, but as building=roof.

Also, when you make a multipolygon, please check that the tags are on it and they aren't left on outer area.

Actually, area:highway=footway implies that they must be connected. Routers won't even try to use it.

There is also highway=pedestrian which is a bit different, but as I've said, not all routers support it.

paths around this
way/1195096150
can be also connected, because not every routing software can plot a route through pedestrian area.

And lane parking should not be attached to the highway

Also parking on the road is not of "surface" type, see
parking=lane

parking=street_side

or it can be mapped directly on the way
osm.wiki/Street_parking

Hi, I usually set

bicycle=yes
instead of
bicycle=designated

if it's a usual zebra.