StationA / tilenol

Scalable, multi-backend geo vector tile server
MIT License
22 stars 6 forks source link

validate layer config zoom values are within min/max bounds #38

Closed brian-dellabetta closed 3 years ago

brian-dellabetta commented 3 years ago

resolves #24

As discussed, layer configurations with minzoom values below the MinZoom server configuration or maxzoom values above the MaxZoom server configuration will trigger a panic at startup.

I added a couple tests and updated the comments describing MinZoom/MaxZoom to add some more clarity, based on conversation thread referenced above.

brian-dellabetta commented 3 years ago

@jerluc I can't assign reviewers on forked pull requests, so I'm linking you in this way :)

jerluc commented 3 years ago

Conflicts have been separately resolved in the updated PR #49 and merged down. Thanks again for your contributions @brian-dellabetta!