Closed
Bug 889985
Opened 11 years ago
Closed 11 years ago
Consolidate es_app_to_dict and app_to_dict
Categories
(Marketplace Graveyard :: Code Quality, defect, P3)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 945820
People
(Reporter: andy+bugzilla, Unassigned)
Details
I think there's a lot of common in those two methods, I wonder if we can consolidate them down into one method. Plus, the coverage of es_app_to_dict worries me.
Reporter | ||
Comment 1•11 years ago
|
||
This is in mkt/webapps/utils.py btw.
Updated•11 years ago
|
Priority: -- → P3
Comment 2•11 years ago
|
||
Possibly moving from tastypie to DRF (bug 910623 among others) will allow us to get rid of most of it, by converting it to a serializer class that can then be inherited to support the ES variant. Bonus: better creation/edition support.
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•