aharvey's Comments
| Changeset | When | Comment |
|---|---|---|
| 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 | |
| 75245237 | about 6 years ago | It would be great to documented this tag on the wiki. Currently the only documentation is for the tag you removed osm.wiki/Proposed_features/aerobridge |
| 75305781 | about 6 years ago | When using a dual name "name:Mrs Macquaries Point / Yurong", then we need "name:en" to equal "Mrs Macquaries Point" so you can break down that dual name into each language. It looks like you've changed that back here http://osmlab.github.io/osm-deep-history/#/node/13816944 |
| 67243224 | about 6 years ago | If you check the history of https://osmlab.github.io/osm-deep-history/#/way/179207161 it looks like you've incorrectly removed the oneway tag, which was then fixed by someone else, and then you removed again, and it was fixed again. Was this a mistake or intentional? |
| 68816300 | about 6 years ago | You've accidentally dragged the bicycle parking see https://osmcha.mapbox.com/changesets/68816300/, however I've fixed this now in changeset/75209417 |
| 75185223 | about 6 years ago | To be honest I couldn't find this documented anywhere on the wiki but now I see https://taginfo.openstreetmap.org/search?q=destination%3Aref%3Alanes is more popular.so I've added some documentation to the wiki. |
| 75204961 | about 6 years ago | reverted see osm.wiki/Spam#SEO_Spam |
| 75204980 | about 6 years ago | reverted see osm.wiki/Spam#SEO_Spam |
| 67461042 | about 6 years ago | Fair point, although I wouldn't consider it a lane, I guess the access:lanes=no implies that it isn't a lane. I've changed it back. |