search

ixmaps / website

IXmaps website (depricated - replaced with website2017 repo)
https://ixmaps.ca
Other
1 stars 0 forks source link

explore - TRdetails missing details #24

Closed Andrew-Clement closed 7 years ago

Andrew-Clement commented 7 years ago

I'm encountering TRdetails screens with many hops missing.

Not sure how this is possible, but it calls attention to the need for a re-write of this.
pastedgraphic-4

colinmccann commented 7 years ago

This is a direct reflection of what is in the database. You can check this out with pgpmyadmin, or see below:

screen shot 2016-10-30 at 4 23 14 pm

Andrew-Clement commented 7 years ago

Thanks Colin, but this begs the question of why such sketchy TRs are getting stored. Even null entries for the missing hops would have been recorded? Or am I missing something?

On 2016-10-30, at 1:23 PM, colinmccann notifications@github.com wrote:

Closed #24.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

colinmccann commented 7 years ago

That's a question for Antonio, unfortunately. I'm not sure about how we decide to accept or reject a route, or how null hops are dealt with...

On Sun, Oct 30, 2016 at 4:27 PM, Andrew-Clement notifications@github.com wrote:

Thanks Colin, but this begs the question of why such sketchy TRs are getting stored. Even null entries for the missing hops would have been recorded? Or am I missing something?

On 2016-10-30, at 1:23 PM, colinmccann notifications@github.com wrote:

Closed #24.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/ixmaps/website/issues/24#issuecomment-257176689, or mute the thread https://github.com/notifications/unsubscribe-auth/AAuEmLsTgPTHQMBJq5aTMDuXJmORFbW7ks5q5P2zgaJpZM4Kkbaz .

Colin