Skip to content

Skip opening hours if no time-zone configured - #4372

Merged
hannesj merged 2 commits into
opentripplanner:dev-2.xfrom
entur:otp2_fix_time_zone
Aug 22, 2022
Merged

hannesj merged 2 commits into
opentripplanner:dev-2.xfrom
entur:otp2_fix_time_zone

Conversation

@hannesj

@hannesj hannesj commented Aug 16, 2022

Copy link
Copy Markdown
Contributor

Summary

Skip Opening hours if no time-zone configured. Log a warning, that timeZone should be added to the build config.

Issue

closes #4355

Unit tests

Migrated

Changelog

The changelog file
is generated from the pull-request title, make sure the title describe the feature or issue fixed.
To exclude the PR from the changelog add the label skip changelog to the PR.

Bumping the serialization version id

If you have made changes to the way the routing graph is serialized, for example by renaming a field
in one of the edges, then you must add the label bump serialization id to the PR. With this label
Github Actions will increase the field otp.serialization.version.id in pom.xml.

@hannesj hannesj added the !Bug Apply to issues describing a bug and PRs witch fixes it. label Aug 16, 2022
@hannesj hannesj added this to the 2.2 milestone Aug 16, 2022
@hannesj
hannesj requested a review from a team as a code owner August 16, 2022 10:12
@hannesj hannesj changed the title Skip Opening hours if no time-zone configured Skip opening hours if no time-zone configured Aug 18, 2022
@hannesj
hannesj merged commit 0d63d21 into opentripplanner:dev-2.x Aug 22, 2022
@hannesj
hannesj deleted the otp2_fix_time_zone branch August 22, 2022 11:44
t2gran pushed a commit that referenced this pull request Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

!Bug Apply to issues describing a bug and PRs witch fixes it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NPE when using OpeningHours with data from OSM

3 participants