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

name for sensors and multiple stations, fix night queries by specifying date filters #6

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

mikeomat
Copy link

@mikeomat mikeomat commented Jan 3, 2024

No description provided.

Tom and others added 2 commits December 27, 2023 09:07
- German Umlaute are received html-escaped for `lastStop`; We unescape
  them to make the readable to the human eye.
Unescape HTML for german umlaute
@mikeomat mikeomat force-pushed the master branch 2 times, most recently from d2fbf6c to a54acc5 Compare January 3, 2024 09:18
added name config to specify sensor names
added dateFilters to retrieve journeys even if there is no next line in the night
@mikeomat mikeomat force-pushed the master branch 2 times, most recently from 47a5c52 to 10d6480 Compare January 3, 2024 14:16
@airman84
Copy link

Hi Mike,

many thanks for your commits, really nice that journeys more than one hour ahead can be pulled now!

But I noticed, that the parameter "additionalTime" seems to don't work anymore? Even with "additionalTime" set, the next sensor get updated just at departure time.

@mikeomat
Copy link
Author

mikeomat commented Jan 13, 2024

Hi, this is because the oebb api is somehow ignoring some of the parameters when you use the date params. i.e. I also needed to fix showJourneys to do it inside the implementation.

@airman84
Copy link

Hi Mike,

do you maybe have any idea why we get duplicate entries around date change? As well some sensors are unknown around midnight.

image

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

Successfully merging this pull request may close these issues.

2 participants