search

harry-s-grewal / mls-real-estate-scraper-for-realtor.ca

Python MLS and Real-Estate Data Scraper for the Realtor.ca Website
MIT License
50 stars 17 forks source link

CurrentPage and MaxRecordsPerPage #4

Open sikanderv opened 1 year ago

sikanderv commented 1 year ago

Hi, thanks for this. Haven't tried this, but I'm using the RapidApi/ApiDojo endpoint for this data.

It looks like the CurrentPage value gets reset to 0 after 50 pages.

And the maximum records per page is 50. Anything above 50 is ignored. Anything below 50 is respected.

Is your experience the same as well?

harry-s-grewal commented 1 year ago

Not for me, I can set records_per_page to 200 before it bottoms out. I just tested it and it works still. Also, current_page doesn't cap out at 50 for me either.

Have you tried seeing your results when using a tool like Postman?

sikanderv commented 1 year ago

It was more of a query than a real issue. I just wanted to know how your app handles this.

By the way, I tried to clone your repo today to test it but met with this error:

image

alex-kozin commented 1 year ago

@sikanderv There is a "Code" button at the top of the repo for you to check the HTTP link to github repo, this worked for me: https://github.com/harry-s-grewal/mls-real-estate-scraper-for-realtor.ca.git

harry-s-grewal commented 1 year ago

^ What he said 😄 I changed the repo name for better SEO