@tomhughes approved this pull request.

Other than the question of the length of the new `admin_level` based levels 
this looks fine to me, and in practice I have yet to see them appear in testing 
as `border_type` takes priority.

I think this is good to go and we can revisit those labels later if they turn 
out to be a problem.



-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5493#pullrequestreview-2669118877
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/5493/review/2669118...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to