djaodjin / djaodjin-saas

Django application for software-as-service and subscription businesses
Other
564 stars 124 forks source link

Roles page #186

Closed knivets closed 5 years ago

knivets commented 5 years ago

So the only problem with this PR is that we don't support multi-column sorting -- that would require to ditch django-extra-views and write our own sorting logic. Let me know what should I do.

smirolo commented 5 years ago

Sort logic for multiple columns already exists and has started to be integrated (ref: #182).

knivets commented 5 years ago

As for the sort logic: the sorting code you linked works completely different from what we have now. Would it make sense to first build the necessary infrastructure (as a standalone PR) and then fix the issue here?