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

Add AccessibilityFeatures to PathLinkStructure, additional AccessFeatures, rename lift to elevator #237

Conversation

ue71603
Copy link
Contributor

@ue71603 ue71603 commented Aug 26, 2022

Additional AccessFeatureTypes and new AccessibilityFeatures attribute.

In the context of PR#234 and #235, I thought it would be good to add accessibility-related data like the following:

PathLink.AccessibilityFeatures: visualSigns, elevatorAccousticAnnouncements, elevatorTactileButtons, etc. This allows for returning accessiblity information attached to PathLinks.

I made AccessFeatureTypes more consistent and complete regarding the possible access equipments by adding singleStep, travelator, etc.

I also changed lift to elevator, to be consistent with NeTEx.

@ue71603 ue71603 added this to the v2.0 milestone Aug 26, 2022
@ue71603 ue71603 mentioned this pull request Aug 26, 2022
@ue71603 ue71603 added enhancement New feature or request documentation labels Aug 26, 2022
OJP/OJP_Trips.xsd Outdated Show resolved Hide resolved
OJP/OJP_Trips.xsd Outdated Show resolved Hide resolved
OJP/OJP_Trips.xsd Outdated Show resolved Hide resolved
Copy link
Contributor

@trurlurl trurlurl left a comment

Choose a reason for hiding this comment

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

I think the typo (AccousticAnnouncements) has to be corrected and the more easily understandable enumerations should be used.

OJP/OJP_Trips.xsd Show resolved Hide resolved
@ue71603
Copy link
Contributor Author

ue71603 commented Oct 18, 2022

André will update PR with the additional input.

trurlurl and others added 3 commits October 20, 2022 09:01
All enums, as discussed in the meeting.
@ue71603 ue71603 requested review from Aurige and trurlurl October 26, 2022 13:43
Copy link
Contributor

@trurlurl trurlurl left a comment

Choose a reason for hiding this comment

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

PR #239 should remain intact when doing the merge.

OJP/OJP_Trips.xsd Outdated Show resolved Hide resolved
trurlurl
trurlurl previously approved these changes Oct 27, 2022
@sgrossberndt sgrossberndt changed the title Added AccessibiliyFeature Type Add AccessibilityFeatures to PathLinkStructure, additional AccessFeatures, rename lift to elevator Nov 7, 2022
OJP/OJP_Trips.xsd Outdated Show resolved Hide resolved
OJP/OJP_Trips.xsd Outdated Show resolved Hide resolved
OJP/OJP_Trips.xsd Outdated Show resolved Hide resolved
@sgrossberndt sgrossberndt merged commit 12b44f7 into changes_for_v1.1 Nov 24, 2022
@sgrossberndt sgrossberndt deleted the Additional-AccessFeature-Types-and-new-Accessibility-Features-attribute-(replaces-#236) branch November 24, 2022 14:42
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