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

Update to SIRI 2.1 using a copy and adapted imports #330

Merged
merged 3 commits into from
Mar 14, 2023

Conversation

ue71603
Copy link
Contributor

@ue71603 ue71603 commented Mar 7, 2023

without copying from upstream to see, if it works....

@ue71603 ue71603 requested a review from skinkie March 7, 2023 09:27
@ue71603 ue71603 added this to the v2.0 milestone Mar 7, 2023
@skinkie skinkie requested a review from sgrossberndt March 10, 2023 13:16
skinkie
skinkie previously approved these changes Mar 10, 2023
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.

I have checked it with XMLlint, works.

@sgrossberndt
Copy link
Contributor

I'd like to have only the necessary xsd sources to be in here, not siris travis integration and examples, I will come up with an updated change for this, but unfortunately this will be after my vacation on 21th of march

@ue71603 ue71603 requested a review from trurlurl March 13, 2023 18:23
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 don't know whether it makes sense to complain about minor problems in external code...

</xsd:annotation>
</xsd:element>
<xsd:element name="AudibleSignalsAvailable" type="AccessibilityStructure" default="false">
<xsd:annotation>
<xsd:documentation>Whether a PLACE / SITE ELEMENT is wheelchair accessible.</xsd:documentation>
<xsd:documentation>Whether a PLACE / SITE ELEMENT is wheelchair accessible. Default is 'false'.</xsd:documentation>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<xsd:documentation>Whether a PLACE / SITE ELEMENT is wheelchair accessible. Default is 'false'.</xsd:documentation>
<xsd:documentation>Whether a PLACE / SITE ELEMENT has Audible signals for the visually impaired. Default is 'false'.</xsd:documentation>

<xs:enumeration value="tournament"/>
<xs:enumeration value="tradeFair"/>
<xs:enumeration value="waterSportsMeeting"/>
<xs:enumeration value="wineFestival"/>
<xs:enumeration value="winterSportsMeeting"/>
<xs:enumeration value="other"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

politicalDemonstration missing? Carnival and Alpabzug would be procession? :-)

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 in html dok seams to be fine.

@skinkie skinkie merged commit 241cf4b into changes_for_v1.1 Mar 14, 2023
@skinkie skinkie deleted the Siri_2.1_copied branch March 14, 2023 16:05
@sgrossberndt sgrossberndt changed the title copied SIRI 2.1 and adapted imports Update to SIRI 2.1 using a copy and adapted imports Mar 21, 2023
@trurlurl
Copy link
Contributor

@ue71603 Does this need some update to the documentation (the documentation labe is set)?

@ue71603
Copy link
Contributor Author

ue71603 commented Apr 21, 2023

@trurlurl

  • If it is mentioned somewhere in the documentation which SIRI is used, it should be adapted.
  • If we use SIRI tables in chapter 8 perhaps a brief check is necessary, if it still fits.
    Otherwise I think that the real changes will manifest in other PR and the documentation comes from there.

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

Successfully merging this pull request may close these issues.

5 participants