Fixed issue #28 - incorrect option chain fields.

xemwebe / yahoo_finance_api

Simple wrapper to yahoo! finance API to retrieve latest quotes and end-of-day quote histories

Apache License 2.0

64 stars 29 forks source link

Fixed issue #28 - incorrect option chain fields. #29

Closed avhz closed 1 year ago

avhz commented 1 year ago

I patched up the YOptionResults::scrape method.

The parsing works properly (had to filter out commas and % signs, etc), and the fields are the correct ones now.

xemwebe commented 1 year ago

Thanks for pointing to this error and providing the fix. Apparently the implementation was still stuck in a POC state. I will remove the println!, though.