reply on: One of those Japanese countries \ stacker news ~AskSN

pull down to refresh

192 sats \ 8 replies \ @0xbitcoiner 9h \ on: One of those Japanese countries AskSN

This post’s got a 'bug' when sorting items by ‘recent’. It doesn’t sort here, only works after you click ‘recent’ and hit refresh (F5). After the items are sorted, new comments don’t show up at the top. cc/ @k00b @ek @sox

33 sats \ 6 replies \ @sox 9h

I can reproduce here, but not in dev. Let's see what's causing this.

For now one clue is that this only happens in client-side navigation: click on a comment -> from the top SN bar go back -> sorts stop working

edit: only here

0 sats \ 1 reply \ @0xbitcoiner 6h

Weird, the 'recent' filter is already working.

63 sats \ 0 replies \ @sox 5h

This confirms my suspects on pagination. When there are a lot of comments, but we didn’t reach the threshold for pagination, comments get duplicated (2-3 times).

Thanks for noticing, I’m working on a fix ^^

0 sats \ 3 replies \ @OneOneSeven 7h

Am I tripping or are a lot of the comments on this thread duplicated lol??

0 sats \ 2 replies \ @sox 7h

No, you’re not tripping…

Seems there are some issues with pagination that are related with the issue @0xbitcoiner reported. I’ll open a github issue in the meantime.

0 sats \ 0 replies \ @SimpleStacker 6h

Sorting my recent worked on desktop but not on mobile for me just fyi

0 sats \ 0 replies \ @Undisciplined OP 6h

I've never broken Stacker News before

That should earn me a badge

0 sats \ 0 replies \ @Undisciplined OP 9h

Thanks for the tip