Xvtn's Comments
Changeset | When | Comment |
---|---|---|
147599015 | over 1 year ago | Hi! Since you requested a review, I looked over your changes here. Looks good to me, no complaints! Thanks for your contributions! |
147863240 | over 1 year ago | Hi! Since you requested a review, I looked over your changes. Looks great to me, no issues I can see! Thanks for your contributions. |
147787141 | over 1 year ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your changes here. Looks great to me, no complaints. Thanks for your contributions! |
147838302 | over 1 year ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your changes here. Looks great overall - The only suggestion I'd give is to check the context as far as building outlines. In this case, looking at the roof, it looks like this is all one building structure, containing multiple businesses. Therefore I'd recommend keeping businesses as single nodes placed within the building. (As opposed to overlapping, conflicting building ways - not recommended.)
|
147915279 | over 1 year ago | Hi! Since you requested a review, I looked over your changes here. Everything looks good to me, no complaints! Thanks for your contributions! |
147916958 | over 1 year ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your changes here. Everything looks great, except that building:levels shouldn't be 0. It's my understanding that building:levels describes how many above-ground floors there are, so something like this should be at least 1. Let me know if you have any questions, and thanks for your contribution! |
147947525 | over 1 year ago | I went ahead and fixed that house tag issue. |
147947525 | over 1 year ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your changes here. Nice job on these houses - Everything looks great to me except the house=residential tag you've added. I don't think that's a valid value - here's some good info on the use of that tag. osm.wiki/Key:house Other than that, no issues! Let me know if you have any questions, and thanks for your contributions! |
147996236 | over 1 year ago | Hi! Since you requested a review, I looked over your changes here. Everything looks good to me, the only suggestion I have is to remove the semicolon in opening_hours tag. That way, data consumers can parse "temporarily closed" as a comment referring to the `closed` instruction (as opposed to being some additional instruction.) Opening_hours can be really tricky, so for interesting cases like this I'll usually use this tool to make sure I've got it right: https://openingh.openstreetmap.de/evaluation_tool/?EXP=closed%20%22Temporarily%20closed%22&lat=48.7769&lon=9.1844&mode=0# Let me know if you have any questions, and thanks for your contributions! |
147997357 | over 1 year ago | Hi! Since you requested a review, I looked over your changes here. Everything seems good to me. Thanks for your contributions! |
148000998 | over 1 year ago | Great advice by Glassman. If I may add one clarifying thing (if it's not already obvious) - the access=* tag is understood to include (and override) all other access-type tags such as foot, horse, bike, etc. |
148031966 | over 1 year ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your changes here. Looks good to me! Thanks for your contribution. |
147522988 | over 1 year ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your changes. Everything looks good - my only suggestion is that you should save your work before moving on to another area. When you edit in two completely different areas, the bounding box is huge. Let me know if you have any questions, and thanks for your contributions! |
147522988 | over 1 year ago | Привет и добро пожаловать в OpenStreetMap! Поскольку вы запросили проверку, я просмотрел ваши изменения. Все выглядит хорошо — мое единственное предложение — сохранить свою работу, прежде чем переходить к другой области. Когда вы редактируете две совершенно разные области, ограничивающая рамка становится огромной. Дайте мне знать, если у вас есть какие-либо вопросы, и спасибо за ваш вклад! (Машинный перевод с английского) |
148045482 | over 1 year ago | Hi! In the future, do you mind providing a descriptive changeset comment? That helps other mappers review your work. Here's some more info on good comments: osm.wiki/Good_changeset_comments |
147707615 | over 1 year ago | Hi! Since you requested a review, I went over your changes here. Looks good to me, no complaints! Thanks for your contributions! |
147729281 | over 1 year ago | Hi, and welcome to OpenStreetMap! It looks like this feature (Body Rock Sculpting Med Spa) might be misplaced or spam, based on the street address. Can you double check it? |
147738658 | over 1 year ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your changes. Everything looks good to me. And thanks for providing a descriptive changeset comment and source!
|
147993300 | over 1 year ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your changes. I think the operation you are looking for here is "Disconnect". In the iD (online OSM) editor, you can select a feature and disconnect it so that you can move it independently of other features. I went ahead and fixed the geometry of the building. Let me know if you have any questions! Thanks for your contributions!! |
147698571 | over 1 year ago | Here's a wiki article with more info on stop signs: osm.wiki/Tag:highway%3Dstop |