-
Notifications
You must be signed in to change notification settings - Fork 39
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
Fix annotation errors and align with definitions in NeTEx pt 3 #383
Fix annotation errors and align with definitions in NeTEx pt 3 #383
Conversation
@@ -181,12 +181,12 @@ | |||
<xsd:choice minOccurs="0"> | |||
<xsd:element name="ExchangableFromAnyTime" type="EmptyType"> | |||
<xsd:annotation> | |||
<xsd:documentation>Special value Can be exhanged from Purchase onwards.</xsd:documentation> | |||
<xsd:documentation>Special value - Can be exchanged or refunded from any point after purchase.</xsd:documentation> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would just say exchanged here (for consistency)
</xsd:annotation> | ||
</xsd:element> | ||
<xsd:element name="ExchangableFromDuration" type="xsd:duration"> | ||
<xsd:annotation> | ||
<xsd:documentation>Duration to start of period before (negative) or after (positive) the trigger point (i.e. either Start of Validity or First Use ) or that ticket.</xsd:documentation> | ||
<xsd:documentation>Duration to start of period before (negative) or after (positive) the trigger point (i.e. either Start of Validity or First Use ) after which ticket may be exchanged refunded or resold.</xsd:documentation> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would just say exchanged here (for consistency)
</xsd:annotation> | ||
</xsd:element> | ||
<xsd:choice minOccurs="0"> | ||
<xsd:element name="ExchangableUntilAnyTime" type="EmptyType"> | ||
<xsd:annotation> | ||
<xsd:documentation>Can be exchanged anyTime.</xsd:documentation> | ||
<xsd:documentation>Can be exchanged, refunded or resold up until any point in time after purchase.</xsd:documentation> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would just say exchanged here (for consistency)
</xsd:annotation> | ||
</xsd:element> | ||
<xsd:element name="ExchangableUntilDuration" type="xsd:duration"> | ||
<xsd:annotation> | ||
<xsd:documentation>Duration to end of period before (negative) or after (positive) the trigger point (i.e. either Start of Validity or First Use ) or that ticket.</xsd:documentation> | ||
<xsd:documentation>Duration to end of period before (negative) or after (positive) the trigger point (i.e. either Start of Validity or First Use ) after which ticket may be exchanged refunded or resold.</xsd:documentation> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would just say exchanged here (for consistency)
@@ -242,12 +241,12 @@ | |||
<xsd:sequence> | |||
<xsd:element name="HasFee" type="xsd:boolean" minOccurs="0"> | |||
<xsd:annotation> | |||
<xsd:documentation>Whether othere is a fee for reselling.</xsd:documentation> | |||
<xsd:documentation>Whether these is a fee for a refund, exchange or reselling.</xsd:documentation> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would just say exchanged here (for consistency)
I would favour an harmonised wording: either "exchange" either " resold (ie refunded or exchanged respectively)" but always have the same one |
@Aurige |
Commit only contains cleanup of xsd:documentation
See NeTEx part 3 chapter 7.6.1.3.1.2 Reselling – Model Element and subchapters / underlying element definitions.