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.

Sign up for GitHub

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

Jump to bottom

Refresh park polygon z4 #1515

Merged
merged 1 commit into from Mar 23, 2023
Merged

Refresh park polygon z4 #1515

merged 1 commit into from Mar 23, 2023

Conversation

benedikt-brandtner-bikemap
Copy link
Contributor

Refresh osm_park_polygon_dissolve_z4 materialized view after update in update_park_polygon.sql.

See #1433 for more detailed discussion.

@ZeLonewolf
Copy link
Contributor

Was it intended that this PR pulled in the transportation layer changes?

@benedikt-brandtner-bikemap
Copy link
Contributor Author

Hey, it was part of the same branch containing all changes discussed in #1433 which i based ontop of #1505 but i can rebase directly ontop of master as well if that would be better for you.

@ZeLonewolf
Copy link
Contributor

I assume it would be easier for whoever reviews it :)

@benedikt-brandtner-bikemap
Copy link
Contributor Author

Have rebased it

@TomPohys
Copy link
Member

Thanks a lot! Could you please rebase it?

@benedikt-brandtner-bikemap
Copy link
Contributor Author

Have rebased it again

@TomPohys
Copy link
Member

Awesome! Thank you very much!

@TomPohys TomPohys merged commit f3dbb0a into openmaptiles:master Mar 23, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TomPohys TomPohys TomPohys approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@benedikt-brandtner-bikemap @ZeLonewolf @TomPohys