Skip to content

bugfix(#53): improve hierarchy handling in ImportService and add deta… - #54

Merged
dgraf-gh merged 1 commit into
mainfrom
53-missing-admin_level2-boundaries-for-some-countries-us-france-no-country_code-returned
Aug 5, 2026
Merged

bugfix(#53): improve hierarchy handling in ImportService and add deta…#54
dgraf-gh merged 1 commit into
mainfrom
53-missing-admin_level2-boundaries-for-some-countries-us-france-no-country_code-returned

Conversation

@dgraf-gh

@dgraf-gh dgraf-gh commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

…iled admin level validation tests

  • Updated isAdministrativeBoundary to handle additional relation types (multipolygon, multilinestring, route) and improve admin level checks.
  • Enhanced ImportServiceTest with new test cases for boundary hierarchy validation and admin level 2 inclusion.
  • Added logging to HierarchyCache for better error reporting during boundary and grid candidate fetch operations.
  • Refactored POI search logic for efficiency and reduced redundant checks in tests.

…iled admin level validation tests

- Updated `isAdministrativeBoundary` to handle additional relation types (`multipolygon`, `multilinestring`, `route`) and improve admin level checks.
- Enhanced `ImportServiceTest` with new test cases for boundary hierarchy validation and admin level 2 inclusion.
- Added logging to `HierarchyCache` for better error reporting during boundary and grid candidate fetch operations.
- Refactored POI search logic for efficiency and reduced redundant checks in tests.
@dgraf-gh
dgraf-gh merged commit a22a43d into main Aug 5, 2026
1 check passed
@dgraf-gh
dgraf-gh deleted the 53-missing-admin_level2-boundaries-for-some-countries-us-france-no-country_code-returned branch August 5, 2026 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing admin_level=2 boundaries for some countries (US, France) — no country_code returned

1 participant