-
Notifications
You must be signed in to change notification settings - Fork 236
Remove rocketfuel code, keep a small compatibility layer for the API (bug 1106586, bug 1058292) #2762
Conversation
125b0d7
to
b965b6e
Compare
from .managers import PublicCollectionsManager | ||
|
||
|
||
class Collection(ModelBase): |
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.
You're removing the model but I don't see a migration to remove any of the database contents or tables.
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.
Ah, yes, forgot to do that. Added.
r+wc. I'd like to see the migration first. |
Some interesting test failures, looking into them now. |
7c83c7e
to
0649803
Compare
Fixed test failure, added migration and squashed commits. |
@@ -0,0 +1,3 @@ | |||
DROP TABLE `app_collections_curators`; | |||
DROP TABLE `app_collection_membership`; | |||
DROP TABLE `app_collections`; |
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.
Do you need to worry about the translations fields that are related here?
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.
No, it works, it'll just leave orphaned translations, but we should have something else to clean them up.
Looks good to me. |
2a705d6
to
2c0379e
Compare
…(bug 1106586, bug 1058292)
Remove rocketfuel code, keep a small compatibility layer for the API (bug 1106586, bug 1058292)
https://bugzilla.mozilla.org/show_bug.cgi?id=1106586
https://bugzilla.mozilla.org/show_bug.cgi?id=1058292