finvizfinance
finvizfinance copied to clipboard
fixed limit not actually working in overview.py
Description
limit param for Overview screener would not work and scraper would grab all items. Now actually stops at closest multiple of 20 above limit as intended. No dependencies needed.
Type of change
- [ ] Bug fix (non-breaking change which fixes an issue)
What you did
Updated logic to correctly limit number of pages to iterate scraper through if a limit is set. Also fixed progress bar to correctly display the page numbers as it scrapes.
Can you also add a test