aharvey's Comments
| Changeset | When | Comment |
|---|---|---|
| 76026455 | about 6 years ago | I moved the remaining tags across from the node to the way now in changeset/76033258 per osm.wiki/Good_practice#One_feature.2C_one_OSM_element |
| 75976516 | about 6 years ago | No worries. As I said I'm not too fused either way. Just adding the religion tag is good, that wasn't the issue, it's about where the amenity=place_of_worship tag should be, should it be on the church building only or on the whole grounds. Either way I think is fine, it's just that the wiki kind of implies it shouldn't be on the grounds. It looks like the amenity=place_of_worship tag is now lacking entirely, so I've added that back in now. When I walked past here today I didn't see a comma in between "...Church Roseville" on the sign, but I'm just nit picking here. |
| 75744794 | about 6 years ago | Please where possible retain the history, I don't see why relation/10162010/history needed to be deleted? You could have just updated the existing relation. |
| 75976516 | about 6 years ago | According to landuse=religious the way I had it before was considered more accepted, but I'm not too fussed either way... |
| 75968088 | about 6 years ago | Are you sure this is not more like a stream than a river, downstream is marked as a stream. (the edit in AU) |
| 75698995 | about 6 years ago | Overall this is okay, but you should separate out the address number from the street, add the +61 prefix to the phone number. |
| 75744794 | about 6 years ago | I've fixed this now in changeset/75746596 |
| 75744794 | about 6 years ago | I don't think this is right, https://www.mapillary.com/map/im/m4NHRN4FjJz-fPJ4zA8EFw. This sign is for cars turning into way/24248082 but your from/to is just Henry Lawson Drive to itself. |
| 75542665 | about 6 years ago | Hi you asked for a review, so I'll provide some feedback. Overall it's good, just keep in mind that a name is only something as signposted or otherwise generally known as, not a description see osm.wiki/Names#Name_is_the_name_only. So "Walking Path" is probably not the name, so best to leave it blank. |
| 59864906 | about 6 years ago | You've added this tag to every node of the way, it looks like you've used select all in JOSM. Could you please fix this? |
| 75305781 | about 6 years ago | I didn't hear back on this so I've updated this again so that name:en is only the English name so that you can distinguish the English name vs. the indigenous name. |
| 5473721 | about 6 years ago | I noticed you've added "Old Bells Line of Road" to https://osmlab.github.io/osm-deep-history/#/way/37627194 on a recent survey I found this one named "Newnes Forest Road" https://www.mapillary.com/map/im/k1mhrLMDQy3dwnWHk-fDkg Do you have any other local knowledge about this road name, I'm trying to decide if I should update it to the signage I saw... |
| 50960319 | about 6 years ago | You've added noname=yes to a bunch of notes along the ways eg. node/4073250116/history I assume this was a mistaken select all in JOSM, just a hint to be extra careful when making mass edits like that. |
| 29445911 | about 6 years ago | Hey I've fixed the helipad you accidentally moved in this changeset changeset/29445911 |
| 50003093 | about 6 years ago | I'm planning on cleaning up this intersection to remove the parallel ways which have no separation apart from paint on the ground, to use lanes and turn:lanes. Turn lanes shouldn't be a separate way they should use turn:lanes instead. |
| 75319160 | about 6 years ago | There's nothing wrong with single ways that enter the roundabout, in fact I'd say this is preferable than splitting the ways for a pedestrian island which I think is not necessary and incorrect. |
| 69845883 | about 6 years ago | Per https://lists.openstreetmap.org/pipermail/talk-au/2019-October/013032.html could you chime into the discussion please? Either here or on the mailing list. |
| 73924464 | about 6 years ago | direction=clockwise doesn't make sense on a lookout, it should be a number/cardinal direction or range. direction=* |
| 75309114 | about 6 years ago | Is this also information=office |
| 75305781 | about 6 years ago |