-
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
Failed validation: #/definitions/QuoteResponseArray #248
Comments
Hey @woophi, thanks for reporting! This is fixed in the devel branch and will be released soon. |
🎉 This issue has been resolved in version 1.14.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
@gadicc have installed the version "yahoo-finance2": "^2.3.0" and still having the same error "message": "must have required property 'circulatingSupply'", you can check code at https://github.com/Sitiapp/stocks-analytics/blob/main/js-functions/functions/wk_nasdaq_eodYFinanceQuotesToGCS.js any suggestion on how to solve this issue? thanks |
Hey @Sitiapp, believe this is unrelated. I know it looks similar because of the first line of the error but that's just by chance... last time we were looking at an index Can you let me know which symbol you're looking up that's failing? If I can reproduce that will be all I'll need, otherwise you'll have to send the full error output. |
@gadicc thanks for your quick reply. Here you can find the stock list i am looping into with "yahoo-finance2": "^2.3.0". |
## [2.3.1](v2.3.0...v2.3.1) (2022-04-13) ### Bug Fixes * **quote:** make customPriceAlertConfidence optional ([#248](https://github.com/gadicc/node-yahoo-finance2/issues/248),[#445](https://github.com/gadicc/node-yahoo-finance2/issues/445)) ([1863fbf](1863fbf)) * **quote:** optional "openInterest" prop; schema + tests ([#248](#248)) ([d8c7808](d8c7808)) * **quote:** optional "typeDisp" prop ([#248](#248)) ([cf7703a](cf7703a))
Looks like you were running into #445, which I've addressed now, and will be in the next release ( I've checked all 1200 and your symbols and they're all working now :) Also found on This will be in the next release and you'll get an automated message here when that happens. Please do report back on your experience. |
🎉 This issue has been resolved in version 2.3.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
@gadicc thank you very much, i tested once and works fine now, i will definitely let you know if i find other issues 👍 |
Great stuff, thanks! |
Validation Error
Minimal Reproduction
const results = await yahooFinance.quote('^VXAPL', {
fields: ['regularMarketPrice', 'regularMarketChange', 'postMarketChange', 'shortName', 'longName'],
});
Symbol(s) that it happened for
Error Message
Environment
Browser or Node: node
Node version (if applicable): v14.15.4
Library version (e.g.
1.10.1
): 1.14.1Additional Context
The text was updated successfully, but these errors were encountered: