hove-io / MttBundle

GNU Affero General Public License v3.0
4 stars 13 forks source link

[Line Timetable ] schedule preview page #61

Open bbonnesoeur opened 9 years ago

bbonnesoeur commented 9 years ago

In order to preview schedule a preview page must be implemented.

notes

vpassama commented 9 years ago

For information, I'm stuck for the moment with a Navitia issue, you can see the @eturck description of the problem : https://github.com/CanalTP/navitia/issues/1033.

To sum up, when I'm parsing each trip notes to make them appear like this :

notes

I can't make distinction between a line note and a trip note because their type is the same and they appear at the same level of description in the route_schedules headers.

For the moment we use a trick consisting in parsing a specific pattern in the note to differentiate them, but I can't merge this code, it's too specific.

For now, I can add all the notes without any distinction but we'll have to modify the code later. Or I don't add any notes and we add them after the issue is solved.

bbonnesoeur commented 9 years ago

@stifoon, are you aware of this problem ? Is there a planned resolution ? thanks !

vpassama commented 9 years ago

I realized we may have a 2nd problem with notes. We have to discuss about it with both teams (Mtt and Navitia).

As you can see in our Navitia response, we have a static code added to the note because we want a specific letter associated to the comment ( m - mercredi for example).

notes

Currently, Mtt generates its own note codes and as you can see in the next screenshot, we introduced our specific need by forcing our notes' codes in the alphabet generation :

notes_stop

So, we should speak about the possibility to add note codes in Navitia or how to manage this need in Mtt.