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

Updated AccessMode from NeTEx and added wheelchair #111

Open
wants to merge 1 commit into
base: integration
Choose a base branch
from

Conversation

ue71603
Copy link
Collaborator

@ue71603 ue71603 commented Apr 14, 2023

some special taxis require a wheelchair to be called.

Aurige
Aurige previously approved these changes Apr 17, 2023
@DohmenClaus
Copy link
Collaborator

Tim will check/approve

@tim-rivett
Copy link
Collaborator

Approved.

@tim-rivett tim-rivett closed this Apr 17, 2023
@ue71603 ue71603 reopened this Apr 17, 2023
@ue71603
Copy link
Collaborator Author

ue71603 commented Apr 17, 2023

@tim-rivett why did you close it? It was not merged. You should approve it with doing a review (press Approve there):
image

tim-rivett
tim-rivett previously approved these changes Apr 17, 2023
@tim-rivett
Copy link
Collaborator

@tim-rivett why did you close it? It was not merged. You should approve it with doing a review (press Approve there): image

I was not concentrating, I should not have done it in the meeting.

@ue71603 ue71603 changed the base branch from master to integration June 21, 2023 22:40
@ue71603 ue71603 dismissed stale reviews from tim-rivett and Aurige June 21, 2023 22:40

The base branch was changed.

@haeckerbaer
Copy link
Collaborator

Discussion on 24.08.2023:
Concerns were raised because AccessModes were added from NeTEx Next branch. Next branch also contains changes that were not approved/final (therefore might not be published in the official documentation). Probably not critical here but still something we have to consider.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants