search

inventaire / inventaire-client

webapp coupled to the inventaire server :books:
https://inventaire.io
48 stars 16 forks source link

Sort by popularity #384

Closed jum-s closed 9 months ago

jum-s commented 1 year ago

solving #114 (5yrs old 🎂 )

The main difficulty was to handle the different sorting by default server side, to be able to recover the initial state of the page. Harmonization server side could have made sense at first sight, but having an agnostic client which store the initial entities sorting felt more future proof, although saving initial server sorting felt like overkill. So instead, the client always treat the entities order as publicationDate sorting by default for entity types that a have wdt:P577 claims, otherwise no sorting is possible by the user (ie. P69 humans)), else the default sorting is by popularity.

jum-s commented 1 year ago

to be merged after #383 [edit: merged]

maxlath commented 1 year ago

@jum-s could you rebase it on master now that #383 is merged?

jum-s commented 1 year ago

its rebased, no conflict

jum-s commented 11 months ago

could also add alphabetical order to the PR

see issue https://github.com/inventaire/inventaire-client/issues/98

jum-s commented 11 months ago

related issue https://github.com/inventaire/inventaire/issues/315