hackernews-TUI icon indicating copy to clipboard operation
hackernews-TUI copied to clipboard

Why the results aren't the same as in the web?

Open ftpd opened this issue 3 years ago • 7 comments

03013cf918d23c53b2c6ad0240a366111f7d66e4

I've decided to give hackernews-TUI a chance today but right after running it I've seen it shows different stories than they are on the web. Why? Do I use it somehow wrong?

ftpd avatar Aug 25 '22 16:08 ftpd

For tabs other than front page, it can either show the stories sorted by date or by points because HN Algolia API is used for that. front page stories are fetched using HN official API. This is why ask stories showed in the app are different from the web.

This can be resolved by using a mix of both APIs for ask,show,etc stories.

aome510 avatar Aug 27 '22 02:08 aome510

Hi @ftpd, I've updated the app in #80 to show shown_hn and ask_hn stories in the same order as in the web. Can you try to the latest build of main to confirm that it works?

aome510 avatar Sep 05 '22 04:09 aome510

Sorry, I was on holidays.

I still see some differences.

ftpd avatar Sep 18 '22 09:09 ftpd

Sorry, I was on holidays.

No worries.

I still see some differences.

Do u have any pictures of such differences? Mine is similar

image

aome510 avatar Sep 18 '22 16:09 aome510

This is how it looks for me: ask

And frontpage looks fine: main

ftpd avatar Sep 18 '22 20:09 ftpd

Looks like you're using the old build of hackernews_tui, as the ask-hn stories are still sorted by date. Can u try it with the latest build from main (the change hasn't been released into a new version):

git clone https://github.com/aome510/hackernews-TUI.git && cd hackernews-TUI
cargo run

aome510 avatar Sep 18 '22 23:09 aome510

Oh, I've just assumed there is a new release ;-) I will build it, but no sooner than today's evening (CEST). I will let you know, sorry for the inconvenience.

ftpd avatar Sep 19 '22 06:09 ftpd

Aaaaaaaand I forgot.

Now I've build from today's master - it works fine. Thanks, I will wait for the new release.

ftpd avatar Oct 03 '22 16:10 ftpd

Aaaaaaaand I forgot.

Now I've build from today's master - it works fine. Thanks, I will wait for the new release.

Great. Thanks for the confirmation. I'll release a new version soon

aome510 avatar Oct 03 '22 17:10 aome510