Remove 'borough', 'macroregion', and 'macrocountry' from the subtype enum#413
Remove 'borough', 'macroregion', and 'macrocountry' from the subtype enum#413tijptjik wants to merge 2 commits intoOvertureMaps:devfrom
Conversation
to be used in the actual releases.
'macrocounty' subtype values
|
@stepps00 Could you take a look at this one? |
|
I'd be inclined to keep the schema as-is, as Overture may still maintain features at these subtypes in future releases. Each are valid representations of administrative divisions, removing them would leave gaps in the subtype hierarchy. That being said, some work could be done to simplify the subtype definitions.. but I think that work would reclassify, rather than remove subtypes. @sasastanojkov thoughts? |
FWIW -- borough and other valid administrative divisions still show up in the |
|
This would likely be superseded by work in #414 |
I personally am ok either way. If we strictly look at what we currently have in the data, I am ok with removing these subtypes. |


Description
It appears that
borough,macroregion, andmacrocountryare no longer valid enum values, as they are no longer in use as of the latest release2025-10-22.0, and the Overture Docs also don't make mention of them anymore.However the schema still had them listed as valid enum values. This PR removed them from the schema.
Reference
Testing
result:
For example, looking up the

subtypeof "The Bronx", which previously was a "borough" and is now a "macrohood"Checklist
Checklist of tasks commonly-associated with schema pull requests. Please review the relevant checklists and ensure you do all the tasks that are required for the change you made.
Abut is not intended to test propertyA's validity, and you made a schema change that invalidates propertyAin that counterexample, fix the counterexample to align it with your schema change.Documentation website
Update the hyperlink below to put the pull request number in.
[Docs preview for this PR.](https://dfhx9f55j8eg5.cloudfront.net/pr/<PUT THE PR # HERE>)