-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add Program Identifier to Support Other Projects #58
Conversation
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 am in favor of this change. We will need to socialize this widely and gather other user input.
Implement this after #64 is merged down. |
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.
A couple comments. Looks good to get started!
+ProgramDate* | ||
+ProgramFundingSource* | ||
+ProgramClassification* | ||
oCampuses/Campus/ |
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.
This needs to be facilities/facilty
Note that this root element sequence refers to the one explained in Proposal #64. | ||
|
||
Steps: | ||
a)Create an element named 'Programs' as a sibling of 'Campuses.' |
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.
Facility
Steps: | ||
a)Create an element named 'Programs' as a sibling of 'Campuses.' | ||
b)Create an element named 'Program' as a child of 'Programs.' | ||
c)Create the elements 'ProgramDate,' 'ProgramFundingSource,' and 'Program Classification' as children of 'Program.' |
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.
Program Classification -> ProgramClassification
@nllong there is not an example file using the program element, do you want me to modify an existing example? if so should that be a new example or just a new version of existing example? |
Yes, i think modifying an existing example will work great. Thanks!
…On Tue, Jan 29, 2019 at 1:13 PM Adriane ***@***.***> wrote:
@nllong <https://github.com/nllong> there is not an example file using
the program element, do you want me to modify an existing example? if so
should that be a new example or just a new version of existing example?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#58 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AB0amusYeyVGy2iBn6QIEMu76S2Oy2Uoks5vIKtMgaJpZM4U2V_X>
.
|
updated program .xsd so that simpletype is referenced and adjusted program element attributes
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.
Looks good!
The BuildingSync schema can be useful and leveraged for more types of project than just audits, and the current schema uses an audit element that can be confusing.