Xvtn's Comments
| Changeset | When | Comment |
|---|---|---|
| 138077298 | about 2 years ago | Hi, thanks for your contribution! Since you requested a review, I looked over your changes. (I know it's late - there seems to be a lack of experienced mappers watching out for new contributors requesting review.) Anyway, no complaints from me, everything looks great! Welcome to OSM! |
| 138077592 | about 2 years ago | Hi, thanks for your contribution! Since you requested a review, I looked over your changes. No complaints from me, looks good. Thanks again and welcome to OSM! |
| 137655229 | about 2 years ago | Hi, thanks for your contribution! Since you requested a review, I looked over your recent changes.
|
| 136846170 | about 2 years ago | Hi, thanks for your contribution! Since you requested a review, I looked over your changes. Looks good to me, no complaints. Welcome to OSM! |
| 144401248 | about 2 years ago | Hi, thanks for your contribution! Since you requested a review, I looked over your changes. - It's best to avoid abbreviations in OSM tags wherever possible. ("N" -> "North")
I went ahead and made those changes. Thanks again, and welcome to OSM! |
| 143628306 | about 2 years ago | Any info on this?
|
| 143252053 | about 2 years ago | Any comments on this?
|
| 139666725 | about 2 years ago | Hi, thanks for your contribution! Since you requested a review, I looked over your changes. Since you included lots of unrelated changes, it's a little hard for me to be completely sure, but I don't see any issues. Thanks again and happy mapping! |
| 139678444 | about 2 years ago | Hi, and thanks for your contribution! Since you requested a review, I went over the changes you made. I think in this case you could use either playground=swing or playground:swing=yes. While playground=swing seems to be much more commonly used, I'd consider that to be more appropriate for an area that only has swings, rather than a larger playground that contains swings as one of many features. playground:swing=yes has the benefit of allowing more feature tags to be added without resorting to multiple values on one key (like playground=swing;slide) which is usually good only as a last resort. Hope that makes sense. Thanks again and happy mapping! |
| 140036750 | about 2 years ago | Hello Vaasan and commenters! I'm going through changesets that have requested a review. Looking over this I'm thinking highway 139/50 going north from exit 15 should not be one-way. But I can't be sure I guess - Can any locals weigh in? |
| 140434171 | about 2 years ago | Hi, and thanks for your contribution! Since you requested a review, I looked over your changes. For the surface tag on that sidewalk, I think the correct one would be concrete rather than concrete_sidewalk. I went ahead and changed that. Thanks again and welcome to OSM! |
| 141003547 | about 2 years ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I went over your changes. Looks great to me, no problems. Thanks for your contributions! |
| 141367051 | about 2 years ago | Hi, and thanks for your contributions! In the future, please use a descriptive changeset comment. An example here might be "Added buildings, fixed Sunset Drive road name, and other minor adjustments". That helps other mappers review what you changed. Thanks again and welcome to OSM! |
| 141388399 | about 2 years ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your recent changes. Looks like someone fixed it already, but in the future it's best to avoid abbreviations in OSM tags, in addresses for example. I also added a more descriptive office=educational_institution tag. Thanks for your contributions! |
| 142214607 | about 2 years ago | Looks like there were already some Wells Fargo tags on the building here. To avoid duplication, I merged them. |
| 142214732 | about 2 years ago | In OpenStreetMap tags, it's best to avoid abbreviations. So for addresses, it should be "West 11000 North" rather than "way/11000 N". That's because it's easy for computers to shorten/abbreviate words, but can be more tricky and error-prone to expand them. Here's some more info: osm.wiki/Abbreviations I fixed the address tag. Thanks! |
| 142214866 | about 2 years ago | Since removing features is somewhat of a more serious action, in the future I'd suggest adding a source for the info when deleting things -- such as an in-person observation, using their website, etc. |
| 142242773 | about 2 years ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I went over your recent changes. Looks great overall, I did leave comments with tips on a couple of your other changesets. |
| 142214915 | about 2 years ago | For this building, at first glance it looks like it's missing a name, but the outer school grounds already are named. IMO, it's best to avoid duplication in situations like that. |
| 142850683 | about 2 years ago | Hi, and welcome to OpenStreetMap! This looks great, no complaints from me. Thanks for your contributions! |