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

Cleanup resort tagging change #2022

Open
nvkelso opened this issue Dec 2, 2021 · 0 comments
Open

Cleanup resort tagging change #2022

nvkelso opened this issue Dec 2, 2021 · 0 comments
Milestone

Comments

@nvkelso
Copy link
Member

nvkelso commented Dec 2, 2021

In #2016 we migrated tourism=resort to leisure=resort but sometimes that's combined with tourism=theme_park so we need to:

  • Make sure we're testing the right thing
  • Do we need to change the order of operations between those to filters in the YAML
======================================================================
FAIL: test_tourism (integration-test.440-zoos-and-other-attractions-tourism.ZoosAndOtherAttractionsTourism)
AssertionError: Did not find feature including properties {'kind': 'theme_park', 'id': -1228099} (because layer 'pois' was empty)
@nvkelso nvkelso added this to the v1.9.0 milestone Dec 2, 2021
@nvkelso nvkelso modified the milestones: v1.9.0, v2.0.0 Apr 29, 2022
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

No branches or pull requests

1 participant