Hi, you've added a few buildings here with "name" (like name=police station way/421862834 ). It's better to add that as a specific feature in iD, rather tahn the name

Hi there, thanks for the contributions.

Careful with the tagging, you need to put a type. And careful with "mini-roundabouts" 😁
way/478786186 & node/4719844782

This looks like a mechanical edit! You changed the `ele` tag of many culverts to a `height` tag. (e.g. way/338049308 ). But the correct tag (instead of `ele`) is `layer`!

Please try to be careful. There is great benefit to actually _looking_ at the data before changing it. 🙂

Hi! You added a lot of `leisure=park` but on nodes. It's best to draw an area for them. 🙂

Careful of the building mapping here! 😁

This looks like a mechanical edit. 😉 If you look at the individual data directly you can sometimes see other mistakes that need correcting.

Like this "building" which is mapped as as a square power line ( way/454413597/history ). 😉

Careful about overlapping roads here. 🙂

You've mapped a few trees in this change and added a teeny little way to each one (e.g. v1 of this way way/525560134/history )

`leisure=park` should be on an area, not a node

You added a few untagged nodes here...

You added a lots of untagged nodes in this changeset....

Hi, in this changeset you have changed a lot of `building=house` to `building=yes` (e.g. way/454858555/history ). Can you explain why you did that? `building=yes` is _less_ accurate than `building=house`. What makes you think these are not houses?

Hello. You added a "service" road right into the scout den ( way/663555985 ). you marked it as `motor_vehicle=yes`, i.e. that one can drive a car along that little stretch. That seems weird to me, is that right?

Hello again! It's great that you want to add historic features to OSM. But we should really try to tag them properly..

Hello. You added Indenpendence Way here ( way/520151921#map=16/10.5208/7.4398 ), is there any reason why the way/road beside it ( way/520151922#map=16/10.5209/7.4397 ) doesn't have a name?

That monument seems a little large, does it really go into the canal? Mapillary implies it's much smaller.

That's what I figured. I changed it to `club=scout` which I think is the correct way to mark such a thing

node/4358905692
changeset/66773832
club=scout

Are you sure this is a camp site? In the middle of Dublin?!

I'm sorry, what? This isn't a HOT mapping task? What's going on?

Welcom to OSM! You asked for a review, so here it is.

This looks good. You've connected the footpath to the road correctly, and added extra details about the path (dirt).

Did you know about the "bird hide" tag ( leisure=bird_hide ) Perhaps you should add that too