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

add the now complete 312 River Run trail #55

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nsteins
Copy link

@nsteins nsteins commented Jun 4, 2023

Added paths for new sections of the 312 River Run trail. Not sure how to verify if I've done the export correctly, so would love some help in making sure it works

Screenshot 2023-06-04 at 2 49 09 PM

@jeancochrane
Copy link
Owner

Thanks for this @nsteins, and thank you for your patience for the past year! I know it's been a while, but do you recall if you reran the underlying OSM import before doing your path editing work? I went to try to recreate it in the prod admin console and noticed that this little segment of path is missing from the OSM data:

Screen Shot 2024-05-14 at 10 10 32 PM

My goal is to do an OSM data refresh sometime before summer, so if it is just a missing way then hopefully the refresh will resolve the issue.

@nsteins
Copy link
Author

nsteins commented May 15, 2024

I don't recall entirely, but I was working with a fresh local build when I submitted these issues, so I imagine I was working with freshly downloaded OSM data from around the time I opened the PR.

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

Successfully merging this pull request may close these issues.

2 participants