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

`Quote` implementing `Serialize` (and `Deserialize`)? #40

Closed meuter closed 4 months ago

meuter commented 4 months ago

Hi,

Would you be open to add Serialize and Deserialize to the list of derived traits on Quote here:

https://github.com/xemwebe/yahoo_finance_api/blob/c967fc8b848f183858feff2cd4c2870724237e67/src/quotes.rs#L127

If so, I would be happy to submit a PR for that.

Thanks in advance for considering this.

EDIT: might as well add Deserialize ;-)

-Cédric

xemwebe commented 4 months ago

Adding Serialize and Deserialize is fine, please feel invited to prepare a PR.

meuter commented 4 months ago

Thanks! PR is submited.