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

Checkin "is on" OSM links broken #15

Open
chadlawlis opened this issue May 31, 2016 · 2 comments
Open

Checkin "is on" OSM links broken #15

chadlawlis opened this issue May 31, 2016 · 2 comments

Comments

@chadlawlis
Copy link

While the direct links to the checkin areas in OSM seem to be functional for checkins that are not on OSM, the direct links for those that are on OSM seem to be "broken".

For example, today I checked into a deli "Aux Délices" in Darien, CT in Swarm which was not yet in OSM. Checkin Checker generated the following direct link: https://www.openstreetmap.org/edit?zoom=19&lat=41.078342&lon=-73.465999. This is the correct area, and I added the deli to OSM.

Yesterday, I checked into "Fort Adams State Park" in Newport, RI in Swarm which was already in OSM. Checkin Checker generated the following direct link: https://www.openstreetmap.org/edit?node=357259588. However, clicking this link took me to the same area as "Aux Délices" (from above) with the resulting link: https://www.openstreetmap.org/edit?node=357259588#map=19/41.07834/-73.46600.

This seems to be happening consistently for me. I have cleared my cache and done a hard refresh with no success. Simply put, the direct link to places already in OSM ultimately navigate to the previous area from a checkin not in OSM.

Any idea what's going on here @iandees? Let me know if this needs further clarification/examples.

@iandees
Copy link
Member

iandees commented May 31, 2016

Huh, interesting. I guess a fix here would be to include the lat/lon in the edit link. I was expecting OSM to automatically do that based on the node position.

@chadlawlis
Copy link
Author

chadlawlis commented May 31, 2016

That could work. Another fix would be to reformat the link to the node itself, e.g. https://www.openstreetmap.org/node/357259588 (using the same node for "Fort Adams State Park" as above). The user can still choose to edit from here, which generates the following link if iD is selected: https://www.openstreetmap.org/edit?editor=id&node=357259588#map=19/41.47816/-71.33557.

Adding the lat/lon probably makes the most sense though (https://www.openstreetmap.org/edit?node=357259588#map=19/41.47816/-71.33557), as you suggested, keeping it consistent with the links from checkins that are already in OSM.

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

2 participants