FinalsClub / karmaworld

KarmaNotes.org v3.0
GNU Affero General Public License v3.0
7 stars 6 forks source link

clean up CourseListView to have fewer, more standard superclasses #341

Closed btbonval closed 10 years ago

btbonval commented 10 years ago

CourseListView subclasses ListView, ModelFormMixin, and ProcessFormView. This isn't any particular Django recipe I've seen in any docs. It seems pretty non-standard, and it makes the functionality difficult to discern. Plus modifying it can be difficult since the usual fixes found in the usual documentation assume the usual Django workflows. This is a Django workflow, clearly, but it is not a well documented usual workflow.

Ideally this can be simplified to be something more Django-like.

btbonval commented 10 years ago

This is being done (indeed has been done) under the auspice of #294