Skip to content
This repository has been archived by the owner on Mar 9, 2023. It is now read-only.

Fetch Geometry #236

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

Fetch Geometry #236

wants to merge 1 commit into from

Conversation

droogmic
Copy link
Collaborator

No description provided.

@droogmic droogmic marked this pull request as ready for review July 17, 2022 19:36
dabreegster
dabreegster approved these changes Jul 17, 2022
View reviewed changes
Copy link
Contributor

@dabreegster dabreegster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The plumbing looks good to me. Running locally, I can't tell what effect loading the geometry has. If you type in a way ID, the map doesn't autozoom there and show it or anything like that. That's not meant to happen, right?

@droogmic
Copy link
Collaborator Author

The plumbing looks good to me. Running locally, I can't tell what effect loading the geometry has. If you type in a way ID, the map doesn't autozoom there and show it or anything like that. That's not meant to happen, right?

hmm, so I guess it was in draft for a reason then, sort of forgot what I was doing... :P

@droogmic droogmic marked this pull request as draft July 17, 2022 21:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@dabreegster dabreegster dabreegster 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.
2 participants
@droogmic @dabreegster