isso icon indicating copy to clipboard operation
isso copied to clipboard

[feature] Sort by newest/highest votes

Open posativ opened this issue 11 years ago • 9 comments

posativ avatar Sep 06 '13 09:09 posativ

push :)

Schweinepriester avatar Apr 22 '15 08:04 Schweinepriester

+1!!

georg90 avatar Jan 10 '16 15:01 georg90

+1

lost-carrier avatar May 07 '17 13:05 lost-carrier

#322 tried to implement it but it fails due to sorting happening on client side, and pagination retrieval on the server side. It needs a bit more work.

blatinier avatar Sep 02 '17 12:09 blatinier

A quick and dirty workaround: add 'asc': 0 here:

https://github.com/posativ/isso/blob/1633d9261ec9877358cda7acc7d78ecb9b09562c/isso/views/comments.py#L752#L757

angristan avatar Feb 11 '20 10:02 angristan

plz somebody implement this as soon as possible. because specially when you limit the default comment numbers to be shown , it is very logical that the latest ones should be shown first.

mdbaniani avatar Dec 20 '21 15:12 mdbaniani

A quick and dirty workaround: add 'asc': 0 here:

https://github.com/posativ/isso/blob/1633d9261ec9877358cda7acc7d78ecb9b09562c/isso/views/comments.py#L752#L757

How exactly you do that? I use docker. (it finally works, but oldest first is a dealbreaker for me) - to asnwer my own question:

Copy comments.py to current folder outside docker docker cp isso_xyz:/isso/isso/views/comments.py .

Edit/add 'asc':0 (and remember a comma after the line before) nano works fine

        args = {
            'uri': uri,
            'after': request.args.get('after', 0),
            'asc': 0  # Add this line for newest first sorting / line 852
        }

Copy comments.py back to docker docker cp comments.py isso_xyz:/isso/isso/views/comments.py

docker restart <container name or ID>

That's it!

4044ever avatar Feb 20 '24 12:02 4044ever

+1 this. Sad to see this is still an issue after over a decade, even the workaround has problems (replies are sorted in reverse too.)

ggtylerr avatar Apr 01 '24 02:04 ggtylerr

I'm currently working on this feature. I hope to finish the PR soon. I'll link it to this issue.

pkvach avatar Apr 01 '24 05:04 pkvach