-
Notifications
You must be signed in to change notification settings - Fork 0
feature: create base tpdm/edfi_3 models iteration 2 #12
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
base: main
Are you sure you want to change the base?
Conversation
bdd6b79 to
eba633e
Compare
eba633e to
aad2a1e
Compare
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 file is duplicated from what is in PR #10
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.
removed
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 don't think this is needed. person I think is just in the base namespace, I don't think there is a second version in the tpdm namespace. you can double check with the documentation here: https://schema.ed-fi.org/datahandbook-v510/index.html#/Person26772476-74ad-430a-bcb2-f49102f1300f anything that is in the TPDM namespace will have "(TPDM)" at the end of the name. Compare "Person" to "Candidate (TPDM)" as examples.
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.
removed
8403462 to
b27fdcf
Compare
b27fdcf to
e849ae3
Compare
4bd05da to
93b172c
Compare
93b172c to
f9f888d
Compare
No description provided.