akramaznakour / linkedin-scraper Goto Github PK

When using this, the page doesn't actually increment, it scans the same page over and over again.

I don't see anything to actually move page, just to click on cards on a singular page?

Issue:
When the number of search result are huge, e.g. 500 results, usually each page (except the last page) will contain 25 items. However sometimes the scraper found out less than 25 items.

Root cause:
This issue is caused by the "scroll down" operation at each page did not go to the very bottom of that page.
This is because the HTML changes dynamically and load more items when you scroll down. In the code "endScrollTop " is calculated in the first place, the value is no longer valid when you scroll down.
This is the line of code which I talked about:
const endScrollTop = element.scrollHeight - element.clientHeight;
(content/linkedin-scraper.js, line 18)

Suggest Solution:
Move the above line of code into the start of the scrollStep(timestamp) function. So that the "endScrollTop" will be calculated every time in a scroll down operation.
You might also want to increase the time out and duration when calling the simulateRealScrollToEnd() method.

(Forgive me by giving word suggestions instead of code, I created a fork and added some of my own features, and because I am not good at javascript, my code looks pretty mess)