Closed secondaryfun closed 4 years ago
From Zakk on previous PR:
You have it so that a location has a group, but you want that the other way (a group belongs to a location). Under the current setup, there can only be 1 group in a location, but just like in meetup, we want to potentially have lots of groups related to a location.
DONE Can you update the fixtures to reflect these changes to the data? I.e. if a Post has to belong to a group, then can you add a group ID to the posts fixtures?
DONE It looks like the changes from your other PR are in this one too, can you get rid of those changes from this PR? I.e. the changes to Bio.css, Bio.js, Icon.js, TextInputField.js, etc
Error on the original branch. Resubmitted using this branch.
Related Issue (include '#'): https://github.com/CotripperPlatform/CoTrip/issues/649
Description of changes made:
Location: one-to-many relationship to Group added on Location Model Profile: many-to-many relationship to Group added on Profile Model Posts: one-to-many relationship to Group added on Post Model
Is the feature complete/bug resolved/etc..: yes
Any known bugs/strange behavior: no
Is there specific feedback you would like on these changes:
Please check logic for "on_delete":
Post Model - on_delete=models.CASCADE
Location Model - on_delete=models.PROTECT
Profile - blank
Screenshot(s): Profile: