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

Adding SituationFullRef to LIR and SER #268

Merged
merged 8 commits into from
Jan 20, 2023

Conversation

ue71603
Copy link
Contributor

@ue71603 ue71603 commented Dec 3, 2022

Adding SituationRefs to the last two services

  • LocationInformation /PlaceInformation
  • StopEvent

Had to place it a bit strangly for Place, but I guess it works as well.
SER already had SituationRefs for the Calls. I thought it would be good to have it on the StopEvent itself as well.

Fixes: #260

@ue71603 ue71603 force-pushed the SituationRefforLIRandSER branch from b63dba7 to 16d2a1e Compare December 21, 2022 11:31
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.

Changes are OK.

@ue71603 ue71603 requested a review from trurlurl December 28, 2022 16:50
trurlurl
trurlurl previously approved these changes Dec 30, 2022
skinkie
skinkie previously approved these changes Jan 17, 2023
Copy link
Contributor

@sgrossberndt sgrossberndt left a comment

Choose a reason for hiding this comment

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

In general: Please have a look at the generated docs/generated/index.html using https://htmlpreview.github.io/ for each change to make sure everything is working for each change.

@ue71603 ue71603 dismissed stale reviews from skinkie, trurlurl, and AndreasAtSBB via 6f65ee2 January 17, 2023 18:07
@ue71603 ue71603 requested a review from sgrossberndt January 17, 2023 18:12
skinkie
skinkie previously approved these changes Jan 17, 2023
AndreasAtSBB
AndreasAtSBB previously approved these changes Jan 18, 2023
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:

  • ref=SituationFullRef ...
  • name=situationFullRefs ....

skinkie
skinkie previously approved these changes Jan 19, 2023
@sgrossberndt sgrossberndt force-pushed the SituationRefforLIRandSER branch from 14c851d to d1b4053 Compare January 19, 2023 17:53
skinkie
skinkie previously approved these changes Jan 19, 2023
trurlurl
trurlurl previously approved these changes Jan 19, 2023
# Conflicts:
#	docs/generated/index.html
@skinkie skinkie dismissed stale reviews from trurlurl and themself via 83ac438 January 19, 2023 21:03
Copy link
Contributor

@skinkie skinkie left a comment

Choose a reason for hiding this comment

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

@trurlurl you may approve again...

@AndreasAtSBB AndreasAtSBB self-requested a review January 20, 2023 08:06
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.

Now always:
<xs:element name="SituationFullRefs" type="SituationRefList" minOccurs="0">
xs:annotation
xs:documentationA list of references to SITUATIONs.</xs:documentation>
</xs:annotation>
</xs:element>

and HTML doc updated.

@skinkie skinkie merged commit 3df0bd0 into changes_for_v1.1 Jan 20, 2023
@skinkie skinkie deleted the SituationRefforLIRandSER branch January 20, 2023 08:12
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.

6 participants