search

smurfy / fahrplan

QT Application for Public transportation
GNU General Public License v2.0
58 stars 32 forks source link

Fixes for Västtrafik (Sweden) #260

Closed thfi closed 6 years ago

thfi commented 6 years ago

This pull request contains four commits that improve Västtrafik support in Fahrplan. Commit 78e7d10 is most important as it corrects the regular expression extracting the access token from the OAuth authentication. Without it, Västtrafik search does not work. The other three commits are rather small, the only noteworthy change is that of the deviceID generation being changed from a combination of application name, random numbers, and other things to a hash aggregating the current device's network interfaces' hardware names. This allows to identify application instances across restarts. The device identification can be used to get usage statistics from Västtrafik's developer platform.

smurfy commented 6 years ago

Thank you, but, since travis was broken, can you please rebase to latest master (i've just fixed travis there) After travis compiles i will merge the PR.

thfi commented 6 years ago

Given that Västtrafik will conduct its annual time table and route update on December 10, this pull request should get merged and new versions uploaded to various app stores well before that deadline