Skip to content

Conversation

@vzakharko
Copy link

No description provided.

@vzakharko vzakharko force-pushed the feature/create-base-tpdm-models-step2 branch from bdd6b79 to eba633e Compare July 22, 2024 14:25
@vzakharko vzakharko force-pushed the feature/create-base-tpdm-models-step2 branch from eba633e to aad2a1e Compare July 22, 2024 14:33
Copy link
Member

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

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

Copy link
Member

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.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

@vzakharko vzakharko force-pushed the feature/create-base-tpdm-models-step2 branch from 8403462 to b27fdcf Compare July 24, 2024 17:08
@vzakharko vzakharko force-pushed the feature/create-base-tpdm-models-step2 branch from b27fdcf to e849ae3 Compare July 24, 2024 17:10
@vzakharko vzakharko marked this pull request as ready for review July 25, 2024 11:45
@vzakharko vzakharko force-pushed the feature/create-base-tpdm-models-step2 branch from 4bd05da to 93b172c Compare July 25, 2024 13:19
@vzakharko vzakharko force-pushed the feature/create-base-tpdm-models-step2 branch from 93b172c to f9f888d Compare July 25, 2024 17:36
@vzakharko vzakharko requested a review from zaharenka July 30, 2024 13:28
@zaharenka zaharenka removed their request for review September 11, 2024 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants