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

Switching back Place to Location #231

Merged
merged 26 commits into from
Jan 31, 2023
Merged

Conversation

ue71603
Copy link
Contributor

@ue71603 ue71603 commented Aug 11, 2022

This request will give us a lot of headache as it deleted two files. Perhaps we should leave the filenames as Places. What do you think?

Fixes #207

@ue71603 ue71603 added this to the v2.0 milestone Aug 11, 2022
@ue71603
Copy link
Contributor Author

ue71603 commented Aug 11, 2022

will do all examples, when content is ok for group.

@sgrossberndt
Copy link
Contributor

sgrossberndt commented Aug 12, 2022

Git detects the renaming of the files, in that regard it looks good. But I think the current state of this pull request does not reflect all necessary changes. It contains b409827, but not 75c64ff, right?

@sgrossberndt
Copy link
Contributor

sgrossberndt commented Aug 12, 2022

will do all examples, when content is ok for group.

To explain it to the others: The build currently fails as the examples have not been updated yet and thus verification of the examples with the changed xsd fails

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.

See comment in OJP_Locations, line 25.

Documentation: there seem to be deviations from the code from even older changes (e.g. PlaceInformationResult).

OJP/OJP_Locations.xsd Outdated Show resolved Hide resolved
OJP/OJP_Locations.xsd Outdated Show resolved Hide resolved
OJP/OJP_Locations.xsd Outdated Show resolved Hide resolved
OJP/OJP_Locations.xsd Outdated Show resolved Hide resolved
@ue71603 ue71603 requested review from trurlurl and Aurige September 12, 2022 09:07
@trurlurl
Copy link
Contributor

Documentation updated.

trurlurl
trurlurl previously approved these changes Sep 12, 2022
trurlurl
trurlurl previously approved these changes Sep 19, 2022
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.

Just a minor point I stumbled across, see comment line 58 in OJP_Places.

OJP/OJP_Locations.xsd Outdated Show resolved Hide resolved
OJP/OJP_JourneySupport.xsd Outdated Show resolved Hide resolved
OJP/OJP_JourneySupport.xsd Outdated Show resolved Hide resolved
OJP/OJP_Locations.xsd Outdated Show resolved Hide resolved
OJP/OJP_Locations.xsd Outdated Show resolved Hide resolved
trurlurl
trurlurl previously approved these changes Nov 24, 2022
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.

No problem from my (limited) perspective.

github-actions bot and others added 13 commits January 27, 2023 14:52
Co-authored-by: Stefan de Konink <stefan@konink.de>
Co-authored-by: Stefan de Konink <stefan@konink.de>
Co-authored-by: Stefan de Konink <stefan@konink.de>
Co-authored-by: Stefan de Konink <stefan@konink.de>
Co-authored-by: Stefan de Konink <stefan@konink.de>
Co-authored-by: Stefan de Konink <stefan@konink.de>
Co-authored-by: Stefan de Konink <stefan@konink.de>
@ue71603 ue71603 force-pushed the Location_back_to_Place branch from 4c68af4 to 388bb65 Compare January 27, 2023 13:52
Copy link
Contributor Author

@ue71603 ue71603 left a comment

Choose a reason for hiding this comment

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

updated the mistake

@@ -125,10 +125,10 @@
<xs:documentation>Problems related to this LOCATION result.</xs:documentation>
</xs:annotation>
</xs:element>
<xs:element name="Place" type="PlaceStructure"/>
<xs:element name="Location" type="PlaceStructure"/>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

studidity, I guess ;-)

@skinkie
Copy link
Contributor

skinkie commented Jan 28, 2023

@sgrossberndt if you like, you can merge.

@sgrossberndt sgrossberndt merged commit 448bcf2 into changes_for_v1.1 Jan 31, 2023
@sgrossberndt sgrossberndt deleted the Location_back_to_Place branch January 31, 2023 10:29
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.

Change from place to location and adding further HTML doc.

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.

8 participants