Xvtn's Comments
| Changeset | When | Comment |
|---|---|---|
| 144679046 | Oh, I'm not sure how it works in your system, AFAIK checking the box just adds review_requested=yes tag to your changeset, which anyone can view. I'm using a tool called OSMCha to review changesets with that tag. (I'm just some rando.) But perhaps your OSM.us tasks setup has people reviewing all changes specific to that too? |
|
| 143439615 | I went ahead and made that change. |
|
| 143439615 | For situations like this where it's clear that parking is operated by two separate businesses, I think it's best to make them separate lots. (That way someone can come through later and give them potentially different tags like access=* or name=*.) |
|
| 143441367 | Looks great! Thanks for contributing. See my note on your other changeset regarding multipolygons. |
|
| 143443437 | For areas with voids in them like this parking lot, you can use a multipolygon. If you don't know already, they're features that have some number of outer and inner members. I made one with your original lot outline as the outer, and some other lines as inner. An easy way to create these in iD is to draw the outer, tag it as appropriate, then use the Line tool to draw the inner. Then select both of them and press "C". That'll auto-create a relation and move the tags over. |
|
| 143443694 | Looks great. Thanks for your contribution! |
|
| 143532851 | Looks great! Thanks for your contributions. |
|
| 143817754 | It looks like this is either severely misplaced or spam. Zuri, can you give some more info or comment on that? |
|
| 143933492 | Actually, looking a little closer, it seems like this might be blogspam. The website provided seems a little fishy to me. Can you provide any info, Nora? |
|
| 143933492 | Hi, thanks for your contribution! Since you requested a review, here are a few tips: - 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! |
|
| 144154522 | Looks great! Thanks for your contributions. I added some other details including the car wash. Re: outdated info, it looks like these buildings are pretty new, but some areas are more up-to-date than others. It all depends on how active local mappers like you or me are! |
|
| 144432425 | Looks good overall. Thanks for your contributions! Speed limits are valuable info for routing.
|
|
| 144488802 | Looks great. Thanks for your contributions!! I also tidied up a couple of nearby roads. |
|
| 144488840 | Looks good. Thanks for your contributions! |
|
| 144679046 | Looks great. Thanks for your contributions! (Just out of curiosity, looks like you're a fairly experienced mapper - Did you request a review hoping for someone with more authority to go over it? Or just wanted a second pair of eyeballs?) |
|
| 144866328 | Hi, thanks for your contribution! Since you requested a review, here are a few tips: - 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! |
|
| 145020280 | Looks great. Thanks for your contribution! One minor thing - changeset comment is for describing what you changed. So in the future you might put something like "Added buildings in Heber and surrounding area".
|
|
| 145051283 | Hi, thanks for your contribution! Since you requested a review, here are a few tips: - In the iD editor, (the one you used) you can select an area and hit "Q" to square up its corners. It's surprising how much nicer it can make things look!
I went ahead and made those changes. Thanks again, and welcome to OSM! |
|
| 145053920 | Also, I'm about 95% certain I'm correct, but feel free to challenge me on that if you disagree. :) |
|
| 145053920 | Hi, and thanks for your contribution! Looks great overall. Since you requested a review, I went over it and noticed a minor thing. There was one redundant traffic light node such that a northbound driver on Friant Rd would have to pass through two such nodes. I removed it. Thanks again! |