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

clean up the API #76

Open
2 of 4 tasks
derhuerst opened this issue Aug 22, 2018 · 20 comments
Open
2 of 4 tasks

clean up the API #76

derhuerst opened this issue Aug 22, 2018 · 20 comments
Assignees
Labels
breaking breaking change
Milestone

Comments

@derhuerst
Copy link
Member

derhuerst commented Aug 22, 2018

  • remove departure.trip & movement.trip – They have a confusing name and may not even be the trip nr all the time. – 1269773
  • split departure.line into dep.trip & dep.trip.linedep.line.id/arr.line.id & dep.line.fahrtNr/arr.line.fahrtNr don't make sense. id (as it currently is) and fahrtNr are properties of the trip/vehicle, not the line.
  • addresses have an ID too – find a way to differentiate them from POIs – addresses have an ID too – find a way to differentiate them from POIs #42
  • rename journey.refreshToken to journey.id
@derhuerst derhuerst added the breaking breaking change label Aug 22, 2018
@derhuerst derhuerst changed the title split departure.line into dep.trip & dep.trip.line clean up the API Aug 27, 2018
@derhuerst
Copy link
Member Author

derhuerst commented Sep 2, 2018

  • leg.mode = 'walking' implies that leg.mode always contains the mode of the line of the leg, but it doesn't. – 02c95ff

@derhuerst
Copy link
Member Author

derhuerst commented Sep 2, 2018

  • rename leg.id to leg.tripId – 8e0b334

@derhuerst
Copy link
Member Author

derhuerst commented Sep 3, 2018

  • remove or rename line.productCode8dd4ef5
  • remove or rename line.class – 8dd4ef5

@derhuerst
Copy link
Member Author

derhuerst commented Sep 17, 2018

  • rename station(id) to stop(id) – 94fb64d

@derhuerst
Copy link
Member Author

derhuerst commented Sep 24, 2018

@derhuerst
Copy link
Member Author

derhuerst commented Sep 25, 2018

  • remove profile.journeysNumF (51f4a66) – ÖBB needs it.
    • hafas-client/index.js

      Lines 172 to 177 in 129caa7

      // With protocol version `1.16`, the VBB endpoint fails with
      // `CGI_READ_FAILED` if you pass `numF`, the parameter for the number
      // of results. To circumvent this, we loop here, collecting journeys
      // until we have enough.
      // see https://github.com/public-transport/hafas-client/pull/23#issuecomment-370246163
      // todo: check if `numF` is supported again, revert this change
    • if (journeys.length >= opt.results) { // collected enough

@derhuerst
Copy link
Member Author

derhuerst commented Oct 8, 2018

  • profiles: rename nationalExp -> nationalExpress – c13138c

@derhuerst
Copy link
Member Author

derhuerst commented Oct 13, 2018

  • formerScheduled{Arrival|Departure}Platform -> scheduled{Arrival|Departure}Platform – 969c045

@derhuerst
Copy link
Member Author

derhuerst commented Oct 25, 2018

  • movement.nextStops -> movement.nextStopovers – b4d763f

@derhuerst
Copy link
Member Author

derhuerst commented Oct 25, 2018

  • remove movement.trip movement.trip -> movement.fahrtNr – 1269773

@derhuerst
Copy link
Member Author

  • leg.mode (from FPTF) vs leg.line.mode
  • leg.public (from FPTF) vs leg.line.public
  • leg.operator (from FPTF) vs leg.line.operator

@derhuerst
Copy link
Member Author

derhuerst commented Nov 12, 2018

  • locations: rename opt.stations to opt.stops

@derhuerst
Copy link
Member Author

derhuerst commented Nov 21, 2018

  • rename profile.formatStation/profile.parseStation -> profile.formatStop/profile.parseStop

@derhuerst
Copy link
Member Author

derhuerst commented Nov 21, 2018

  • rename opt.stationLines -> opt.linesOfStops

@derhuerst
Copy link
Member Author

derhuerst commented Jun 21, 2019

  • trip.line.fahrtNr -> trip.fahrtNr, leg.line.fahrtNr -> leg.fahrtNr
  • add trip.name to differentiate it from trip.line.name

@derhuerst
Copy link
Member Author

derhuerst commented Dec 10, 2019

  • movement.location -> movement.position

@derhuerst derhuerst modified the milestones: 5, 6 Jan 5, 2020
@derhuerst
Copy link
Member Author

  • move profile = {...defaultProfile, ...profile} from index.js/throttle.js/retry.js to p/*/index.js

@derhuerst
Copy link
Member Author

derhuerst commented Feb 11, 2020

@derhuerst
Copy link
Member Author

@derhuerst derhuerst self-assigned this Dec 30, 2021
@derhuerst
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking breaking change
Development

No branches or pull requests

1 participant