finvizfinance icon indicating copy to clipboard operation
finvizfinance copied to clipboard

fixed limit not actually working in overview.py

Open AdamMuzz opened this issue 1 year ago • 1 comments

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.

AdamMuzz avatar Dec 26 '23 00:12 AdamMuzz

Can you also add a test

tli26 avatar Feb 16 '24 22:02 tli26