Skip to content

Remove required 'id' attribute from accessibility schema#977

Merged
TuThoThai merged 1 commit intonextfrom
id_accessibilitylimitation
Oct 25, 2025
Merged

Remove required 'id' attribute from accessibility schema#977
TuThoThai merged 1 commit intonextfrom
id_accessibilitylimitation

Conversation

@TuThoThai
Copy link
Copy Markdown
Collaborator

Based on the merge of #970, I have looked more into details in elements that I believe do not need an "id" since they are not referened anywhere else.
One element addressed in this PR is "AccessibilityLimitation". As it is embedded within an physical item for which it describes its accessibility, I believe that having an "id" in the first place was a bug in the XSD. It does not appear in the documentation.

Removed the required 'id' attribute from the element "AccessibilityLimitation"
@TuThoThai TuThoThai added the bug Technical mistake, inconsistency with the documentation, etc. label Oct 23, 2025
@TuThoThai TuThoThai added this to the netex_2.0 milestone Oct 23, 2025
@skinkie skinkie self-requested a review October 24, 2025 16:58
@TuThoThai
Copy link
Copy Markdown
Collaborator Author

@ue71603, @trurlurl or @thbar: may I get your review on this one?

@TuThoThai
Copy link
Copy Markdown
Collaborator Author

Merged based on confirmation that id is not needed for AccessibilityLimitation as not referenced elsewhere

@TuThoThai TuThoThai merged commit 1dfb32a into next Oct 25, 2025
1 check passed
@TuThoThai TuThoThai deleted the id_accessibilitylimitation branch October 25, 2025 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Technical mistake, inconsistency with the documentation, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants