-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
I get an error when querying { fields } in .quotes. #807
Comments
Hey @Emre4x0 Thanks for reporting, confirming the bug. Unfortunately this is an issue with the validation framework we use, and I'll need to prepare a proper bug report there. I hope I'll have a chance over the weekend, let's see. In the meantime, I just wanted to check how much this bug affects you... i.e., do you still get the fields you need without using the Thanks! |
Hello, thank you for ur response. I can get the result I want without using fields. I was using fields to shorten my fetch time. |
I wish you luck with your work @gadicc. I look forward to the new update. 🤩 |
Hey, thanks so much, @Emre4x0, appreciate your super quick response and kind words. I'll leave this open so we can still track it. I'd also like to cleanup the error message A LOT 😅 Thanks again! 🙏 |
Validation Error
Minimal Reproduction
await yahooFinance.quote(symbols.map((symbol) => symbol.name), {
fields: [
"symbol",
"regularMarketChange",
"regularMarketChangePercent",
"quoteType",
"circulatingSupply",
"fromCurrency",
"toCurrency",
"lastMarket",
"coinImageUrl",
"volume24Hr",
"volumeAllCurrencies",
"startDate",
"priceHint"
]
});
Error Message
Environment
Browser or Node: Node
Node version (if applicable): v20.17.0
Library version (e.g.
1.10.1
): 2.12.4Additional Context
when i remove the warning fields, i get different versions of the same error. fetch works when I don't use fields.
The text was updated successfully, but these errors were encountered: