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.

Sign up for GitHub

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

Jump to bottom

Remove PhysicalSystem from Line 71 of CoreConcepts class #43

Closed
ghobona opened this issue May 27, 2020 · 1 comment
Closed

Remove PhysicalSystem from Line 71 of CoreConcepts class #43

ghobona opened this issue May 27, 2020 · 1 comment
Assignees
@arieslai
Projects
CITE
Milestone
0.9

Comments

@ghobona
Copy link
Contributor

ghobona commented May 27, 2020

PhysicalSystem.components has a minOccurs of zero in the schema and the specification document does not require it to be added to a PhysicalSystem element.

Line 71 of the CoreConcepts class should therefore not include PhysicalSystem.
@ghobona ghobona added this to To do in CITE May 27, 2020
@dstenger dstenger moved this from To do to In progress in CITE Jun 3, 2020
@arieslai
Copy link
Contributor

arieslai commented Jun 3, 2020

Fixed in commit #69b1328

@ghobona ghobona closed this as completed Jun 10, 2020
CITE automation moved this from In progress to Done Jun 10, 2020
@arieslai arieslai added this to the 0.9 milestone Aug 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arieslai arieslai

Labels
None yet
Projects
CITE
  
Done
Milestone
0.9
Development

No branches or pull requests
2 participants
@ghobona @arieslai