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

Minor fix for PA US-322 Business (State College). #29

Merged
merged 1 commit into from Aug 13, 2015
Merged

Minor fix for PA US-322 Business (State College). #29

merged 1 commit into from Aug 13, 2015

Conversation

rickmastfan67
Copy link
Contributor

Fixed location for 'US322_W' to re-sync route with mainline US-322 (and I-99/US-220). Also added in one additional point at the most logical road that connects to I-99 Exit #71. See: http://clinched.s2.bizhat.com/viewtopic.php?t=2388&mforum=clinched

Fixed location for 'US322_W' to re-sync route with mainline US-322.  Also added in one additional point at the most logical road that connects to I-99 Exit #71.  See: http://clinched.s2.bizhat.com/viewtopic.php?t=2388&mforum=clinched
@jteresco
Copy link
Contributor

Looks simple enough, going to pull it in.

jteresco added a commit that referenced this pull request Aug 13, 2015
…ollege)-Fix

Minor fix for PA US-322 Business (State College).
@jteresco jteresco merged commit 0e2c483 into TravelMapping:master Aug 13, 2015
@jteresco
Copy link
Contributor

Note that the new line used a different OSM URL format than the one our wpt file format supports. Fixed in master copy, so no worries, but something to keep in mind.

@rickmastfan67
Copy link
Contributor Author

Oops, my bad. Must have striped out that part by accident when taking out the zoom levels.

@rickmastfan67
Copy link
Contributor Author

It doesn't seem that the fix has been added yet to 'master'. So you want me to submit the fix myself?

@jteresco
Copy link
Contributor

You might need to update your copy of the master to match the one on GitHub in the TravelMapping. If not, let's worry about it next week.

@rickmastfan67
Copy link
Contributor Author

I double checked the 'master' before I submitted #31. Last commit was the merge of this ticket with the mistake of mine with the URL string.

I even double checked with what changes were missing between my fork and master. The URL was still the 'messed' up one for the 'VaiBlvd' point. So, we'll deal with #31 once your back. ;)

jteresco pushed a commit that referenced this pull request May 29, 2017
jteresco pushed a commit that referenced this pull request Jan 2, 2018
jteresco pushed a commit that referenced this pull request Sep 10, 2018
Recentering Along US 202, PA 23, PA 29 (Phoenixville), PA 434, PA 179

Also added short segments of PA 44 and PA 434 signed in NY
jteresco pushed a commit that referenced this pull request Nov 20, 2018
jteresco pushed a commit that referenced this pull request Dec 2, 2019
michihdeu pushed a commit that referenced this pull request May 8, 2020
si404 added a commit that referenced this pull request Mar 13, 2023
* Update imna.csv

* Update imna_con.csv

* Update imnb.csv

* Update imnb_con.csv

* Update imn.ttcir.wpt

* Update imn.a006.wpt

* Delete imn.a021.wpt

* Update imn.a018.wpt

* Update imn.a022.wpt

* Update imn.a023.wpt

* Update imn.a043.wpt

* Update imn.a045.wpt

* Update imn.a047.wpt

* Update imn.b001.wpt

* Update imn.b049.wpt

* Update imn.b068.wpt

* Create imn.b072.wpt

* Update imn.b074.wpt

* Update updates.csv
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.

None yet

2 participants