search

clokep / django-querysetsequence

Chain multiple (disparate) QuerySets in Django
https://django-querysetsequence.readthedocs.io/
ISC License
107 stars 25 forks source link

Fixed sort with None order and add test case #97

Closed vuongdv-spinshell closed 1 year ago

vuongdv-spinshell commented 2 years ago

Hi @clokep,

Please, take a look at this PR. As we have discussed in https://github.com/clokep/django-querysetsequence/issues/96 This PR should fix it and shouldn't break anything.

Please, let me know wdyt 🙏 ?

clokep commented 2 years ago

@vuongdv-spinshell I pushed a commit to this branch which should fix this. Can you give it a try and let me know if you see any issues?

clokep commented 1 year ago

Since there's been no response I'm going to assume this fixes the issue appropriately.