-
Notifications
You must be signed in to change notification settings - Fork 168
Remove ability to delete content units #3407
Conversation
pass | ||
|
||
|
||
class CreateReadNamedModelViewSet(mixins.CreateModelMixin, |
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.
Am I the only one that finds this a liiiiittle bit ridiculous?
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.
Yes. Maybe have viewsets extend these mixins directly instead of creating a base viewset for each possible combination?
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.
I'll write up a refactor task which we can discuss next sprint planning
I tried testing this out but I am getting |
@@ -94,11 +94,13 @@ def import_serializers(self): | |||
def import_viewsets(self): | |||
# circular import avoidance | |||
from pulpcore.app.viewsets import (GenericNamedModelViewSet, NamedModelViewSet, |
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.
Man, this code smells real bad.
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.
Agreed
Test failure is unrelated |
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.
👍 tested and it works well.
Update celery stack to v4 so we no longer have to maintain code for both 3 and 4 closes #3407 https://pulp.plan.io/issues/3407
closes #3445
https://pulp.plan.io/issues/3445