models for pivot tables

studycloud / studycloud

A website for sharing student resources, built to benefit generations of future high school and college students.

http://studyclouds.org

11 stars 0 forks source link

models for pivot tables #107

Open aryarm opened 5 years ago

aryarm commented 5 years ago

we don't really use them for a reason: we don't really need them

get rid of

[ ] TopicParent
[x] RoleUser
[ ] ResourceTopic
[x] PermissionRole

but make sure nothing will break when you do this!

aryarm commented 4 years ago

note that we got rid of RoleUser and PermissionRole in #175

aryarm commented 4 years ago

Alternatively, rather than get rid of the TopicParent and ResourceTopic models, we could try to embrace their current function as rarely-used-but-still-eventually-useful code For example, TopicParent is used once in the ResourceTopicTableSeeder And we might want to extend the Pivot class instead of treating it like a regular Model