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)

x86
macOS
defect

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.
This is in mkt/webapps/utils.py btw.
Priority: -- → P3
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.
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.