CDRH / api

Codenamed "Apium": An API to access all public Center for Digital Research in the Humanities resources
https://cdrhdev1.unl.edu/api_frontend
MIT License
3 stars 1 forks source link

facet_sort options #24

Closed jduss4 closed 7 years ago

jduss4 commented 7 years ago

facet_sort right now is assuming count and only allowing override by term. In an effort to make it more backend agnostic, plus user friendly, should switch to use the following:

facet_sort=alpha|direction OR facet_sort=count|direction

Defaults to alpha|desc if nothing is passed in. If one of the above does not match, returns error

jduss4 commented 7 years ago

I'm gonna add that right now the facet sort has some poorly thought out logic going on and a variable that doesn't get used, so, maybe work on that.

jduss4 commented 7 years ago

Ditching alpha because of this issue: https://github.com/CDRH/api/issues/29