cncf / landscapeapp

🌄Upstream landscape generation application
https://landscapes.dev/
Apache License 2.0
255 stars 125 forks source link

Have invalid stock_ticker throw only a normal error, not a fatal error #885

Closed jmertic closed 1 year ago

jmertic commented 1 year ago

I would like to have an invalid stock_ticker value throw only a normal error, not a fatal error, as 99% of the time, we end up adding stock_ticker: null to the entry as a fix. I'd propose that if the stock_ticker truly is invalid, presume it's set to null and don't get the market cap data.

jmertic commented 1 year ago

Fixed by 808d219 - thanks!