-
Notifications
You must be signed in to change notification settings - Fork 664
[personality-insights] SDK overrides content-items' language. #220
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
Comments
I will assign this to nathan since I'm working on the java-skd now |
Oh, that's no good. Let me take a look. |
Actually, this kind of feels like a service bug. But I added a fix (and a couple of tests) to the Node SDK. Do you know if this also affects the other SDKs? |
Oh, and it should be released as v1.3.1 once the CI loop finishes. Can you try it out and confirm that it resolves your issue @arypbatista ? |
Solved my issue @nfriedly . Thanks. I will raise an issue on the service too. Regards |
Cool, thanks! |
When using
node-sdk
's Personality Insights wrapper, and usingcontentItems
as payload without specify a language, the wrapper will assume that the payload is in English.This is a desired thing when sending
text/plain
payload, but if you do the same forapplication/json
payload (content-items), then the service ignores each individual content-item language and tries to process all the content items with the wrapper's language choice.The text was updated successfully, but these errors were encountered: