Welcome to OSM and thanks for contributing to the map. A couple tips for adding additional info to buildings: 1) date something was constructed can be tagged with start_date, 2) you can use old_name for a place like this that used to be a hospital, 3) when tagging building:level on a building outline, include the largest number of levels within the footprint (not a range like 4-5). To add more detail about different parts of a building, see osm.wiki/Simple_3D_buildings Happy mapping!

Hello, and thanks for contributing to OSM! In this changeset, I noticed that in the parking lot of Hosanna Lutheran Church, the highway=service way overlaps itself. It's best to split this into multiple, connected ways. More best practices are here: osm.wiki/Good_practice Happy mapping!

I think that's the convention for areas like this, it's especially useful for routing. More info here: osm.wiki/Sidewalks#Sidewalk_as_separate_way

Hi, I'm wondering why you've disconnected the sidewalks from the streets?

I left a different changeset comment about multipolygon relations--here is a quick-and-dirty how-to: https://help.openstreetmap.org/questions/22542/how-do-i-map-an-island-in-a-lake-using-id-editor-and-the-multipolygon-tag

The residential area with the school inside should be a multipolygon relation, instead of how you have drawn it. The residential outline would be the outer way, and the school outline the inner way, without a connecting line. Send me an OSM message if you're not sure how to do this--it comes in handy!

Hi and welcome to OSM. I notice you've been adding a lot of buildings. One thing that can make them look a little better (and creating them a bit easier) is to use the Square option in the iD Editor: right click on a building, and select it, or click on a building and hit S. Happy mapping!

Hi, is there a reason you deleted the main outer way for the Fort Shafter relation?

Hi and welcome to OSM! I noticed that you have overlapped some of the parking aisles near Ridgeway Center. In general it's best to create some ways that go through an area or form its edges, then fill in with single ways. Here is a link to a QA tool which might help you visualize the issue: http://tools.geofabrik.de/osmi/?view=geometry&lon=-89.86711&lat=35.10545&zoom=17&opacity=0.84 Happy mapping!

Hi and welcome to OSM! Thanks for adding to the map. One hint: I noticed the footway near Spruce and CH 134 overlaps itself. Instead, you would want to create two ways here--for example, one that goes to the NE, and one that connects to it and goes to the NW. Happy mapping!

Hi and welcome to OSM! There are a couple places you have overlapped a way (ie osm.org/way/546438255 and osm.org/way/546438222) In cases like this, please create one way, then start a second that branches off the first instead of making it all one feature. Thanks!

Hi, could you fix multipolygon relation 7757190 so that the outer and inner parts of the building are separate? Let me know if you have questions

Hi, it looks like the building and park may have gotten accidentally connected to the end of West School Street, could you take a look? Also, Morehouse Park should probably be one large area instead of multiple little ones. Park boundaries should generally encompass whatever is in them--roads, buildings, baseball fields, water, etc.

Hi, just an fyi that it's more common to tag fairgrounds as tourism=attraction. See https://overpass-turbo.eu/s/tO7 for examples

Hi, deleting all this seems kind of questionable. By deleting the existing library building, you removed a lot of other information about the library that was in the tags of that way. You also deleted some roof-only sections of the building. You mention fixes to pedestrian areas, but I'm not clear what improvements you made to them. Indoor mapping doesn't always "look good" in a standard rendering but in general if the data is accurate, there needs to be a very good reason to redo/remove it.

Welcome to OSM and thanks for adding to the map! In a strip mall like this, it would be better practice to create a building for the entire structure, and then add nodes for each business within the mall, rather than create individual areas for only some of the stores. Also note, landuse tags should not be used on buildings or stores. Teppanyaki Grill should probably be tagged amenity=restaurant, Duluth Trading should be shop=clothes, etc.

Hi, I notice you previously updated the address of this point, but didn't move its location to the new address--are you planning to do that? Also, you might want to add an office tag if that's appropriate (see osm.wiki/Key:office for values)

Hi and welcome to OSM! I see you flagged "review requested" and these edits look good to me. For businesses, you might also consider adding address, phone #, website, and (if you're feeling adventurous) opening hours. Happy mapping!

Hi, wondering why you created this Edgewood College way that includes the High School and Campus School too? The college is just a part of this area, and they are all independent though related schools. (ie see https://www.edgewood.edu/docs/default-source/aboutdocs/directions/campusmap.pdf)

Hi, it appears you uploaded a bunch of duplicate nodes in this changeset--can you please clean that up?