Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: New endpoint for shapefile data #82

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hepplerj
Copy link
Member

@hepplerj hepplerj commented Apr 3, 2025

This pull request introduces a new feature to handle and serve bills data with parish polygons in GeoJSON format. The primary changes include adding a new handler for this functionality, updating the routes to include the new endpoint, and providing example URLs for the new endpoint in the API documentation.

New feature implementation:

  • bom-shapefiles.go: Added BillsShapefilesHandler to return a GeoJSON FeatureCollection containing parish polygons joined with bills data. This includes dynamic filtering and query execution with a timeout context for performance.
  • bom-shapefiles.go: Introduced the buildSeparateFilters function to construct separate SQL filters for bills and parishes based on URL parameters.

Routing updates:

  • routes.go: Added a new route /bom/bills-geometries to handle requests for bills data with parish polygons.

API documentation:

  • endpoints.go: Updated the EndpointsHandler to include documentation and example URLs for the new /bom/bills-geometries endpoint.

Dependency additions:

  • bom-shapefiles.go: Added imports for strings and time to support the new handler functionality.

@hepplerj hepplerj added the project: bills of mortality Issues for the Bills of Mortality project label Apr 3, 2025
@hepplerj hepplerj self-assigned this Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
project: bills of mortality Issues for the Bills of Mortality project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant