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

Switches for accessibility information #291

Merged
merged 10 commits into from
Jan 20, 2023

Conversation

ue71603
Copy link
Contributor

@ue71603 ue71603 commented Dec 22, 2022

From the meeting: adding the ability (OJP_TripRequest) to specify the accessibility-related information to return as it may differ depending on the user profile and on what data will be used and presented.

@ue71603 ue71603 added enhancement New feature or request documentation labels Dec 22, 2022
@ue71603 ue71603 added this to the v2.0 milestone Dec 22, 2022
AndreasAtSBB
AndreasAtSBB previously approved these changes Dec 23, 2022
Copy link

@AndreasAtSBB AndreasAtSBB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is OK.

@skinkie
Copy link
Contributor

skinkie commented Jan 16, 2023

What I miss in this pull request is the actual 'result structure' for example I want to know what happens if the realtime status would not be available.

@trurlurl
Copy link
Contributor

@skinkie To answer your question as I understand it: IncludeAccessFeatureStatus (in this PR) is used to request or not real time information about access features. As for the result, see PR #238 - there is AccessFeatureStatus (and AccessFeatureStatusText and SituationFullRef) to provide the real time status. If some data is not available, this can be signalled using AccessFeatureWithUnknownAvailability.

Additionally, there is the problem code TRIP_REALTIMEINCOMPLETE that could be used, It should probably also be used by TripRefineRequest, shouldn't it?

# Conflicts:
#	docs/generated/index.html
# Conflicts:
#	docs/generated/index.html
skinkie
skinkie previously approved these changes Jan 20, 2023
skinkie
skinkie previously approved these changes Jan 20, 2023
OJP/OJP_JourneySupport.xsd Outdated Show resolved Hide resolved
@sgrossberndt sgrossberndt merged commit 6702957 into changes_for_v1.1 Jan 20, 2023
@sgrossberndt sgrossberndt deleted the accessibility_information branch January 20, 2023 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc updated enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants