-
Notifications
You must be signed in to change notification settings - Fork 40
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 new taxonomy system #1279
Add new taxonomy system #1279
Conversation
@saschaben please review ERD: |
looks good. please validate independently as well against https://ucsf.box.com/s/9yp0j1xait48n5gqlylvomxtbqextri7 |
per our conversation:
|
* @JMS\ExclusionPolicy("all") | ||
* @JMS\AccessType("public_method") | ||
*/ | ||
class Vocabulary implements VocabularyInterface |
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.
Can't this be a CatagorizableEntity?
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.
it totally can! good catch.
@stopfstedt couple of questions - none of them are breaking. |
… assoc as deprecated.
@jrjohnson fixed all outstanding issues, please review/merge. thanks! |
refs #1257.
this adds the new
vocabulary
andterm
tables, as well as join tables toschool
,session
,course
andprogram_year
.this does not remove the legacy
topic
table and associated code from the system just yet, new and old taxonomy systems will exist in parallel.data migration from topics to vocab/terms are out of scope as well and need to be addressed in a separate issue.