(regression) incorrect number of replies
Closed this issue · 5 comments
The number of thread replies for some posts is incorrect. See screenshots below.
https://tldr.bitcoinsearch.xyz/summary/delvingbitcoin/Feb_2024/combined_Mempool-Incentive-Compatibility#discussion-history
This could have occurred due to parallel scheduling of GitHub Actions. I updated the time for json generation of homepage and newsletter files. I also ran the workflows in order generate the latest homepage.json. I also ran the update submodules action on this repo we the changes gets reflected on the website.
The code seems correct in the backend.
If you take a look at this line in homepage.json file's n_threads` it shows the number of replies.
And the same no. of threads are shown in the discussion history of this post
However on the homepage of Bitcoin TLDR, it is not as per homepage.json.
@Extheoisah , Can you please take a look in the frontend code as well?
found another bug related to replies:
If you take a look here, n_threads is already 0 so it should be the same on the frontend.
Can you please confirm that the frontend team is not preprocessing the value of n_threads (replies) which is being received from homepage.json?
Issue has been resolved here #218