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

definitions added to all requests and responses based on TM #413

Merged
merged 29 commits into from
Oct 16, 2023

Conversation

ue71603
Copy link
Contributor

@ue71603 ue71603 commented Oct 11, 2023

fixes #405

<xs:element name="OJPFareRequest" type="OJPFareRequestStructure" substitutionGroup="siri:AbstractFunctionalServiceRequest"/>
<xs:element name="OJPFareRequest" type="OJPFareRequestStructure" substitutionGroup="siri:AbstractFunctionalServiceRequest">
<xs:annotation>
<xs:documentation>This service provides general, stop-specific and trip-specific fare information. This service implements PI Stop Fare Request, PI FQ Fare Product Request, PI FQ Single Trip Fare Request from TM6.</xs:documentation>
Copy link
Contributor

Choose a reason for hiding this comment

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

I think I want to know what "FQ" is.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh my....

Copy link
Contributor

Choose a reason for hiding this comment

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

Or make it "TM 6" here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<xs:annotation>
<xs:documentation>The Location Information service comprises the following four base functions:
matching text input against possible origin and destination locations, retrieval of all location objects (bunch delivery), geographical context service that provides location objects within a bounding box, reverse address resolution service that delivers the nearest address for a given coordinate.
By means of abstraction these functions are assembled within one single service – the Location Information service. This way, even more possible applications have arisen, such as: finding the nearest stops/stations for a given coordinate, matching text input against the names of locations near a given coordinate. This service implements PI QR Location Request from TM 6.</xs:documentation>
Copy link
Contributor

Choose a reason for hiding this comment

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

QR?

TM 6 -> TM6? (consistency)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This are all packages from TM
image

OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.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.

For me, having the definitions only in the requests would be sufficient, and it would easen maintenance.

OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
@ue71603 ue71603 force-pushed the definitions-for-requests branch from 12225db to 2b50f47 Compare October 15, 2023 17:27
trurlurl and others added 3 commits October 15, 2023 19:59
This just seems to have been missed when resolving conversations...

Co-authored-by: Matthias Günter <[email protected]>
This just seems to have been missed when resolving conversations...
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
OJP/OJP_Requests.xsd Outdated Show resolved Hide resolved
@skinkie skinkie merged commit 1ea44c3 into changes_for_v1.1 Oct 16, 2023
@skinkie skinkie deleted the definitions-for-requests branch October 16, 2023 08:47
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.

3 participants