We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Hi @JanEbbing
I saw you introduced a new DeeplClient classes which is supposed to be used instead of Translator. cf 46b68a7
There is also a DeeplClientOptions class, but I don't see why it extends TranslateTextOptions. Shouldn't it extends TranslatorOptions instead ?
TranslateTextOptions
TranslatorOptions
In the same way I did
new Translator($apiKey, [TranslatorOptions::DEFAULT_MAX_RETRIES => 0])
i would expect now to do
new DeepLClient($apiKey, [DeeplClientOptions::DEFAULT_MAX_RETRIES => 0])
WDYT ?
The text was updated successfully, but these errors were encountered:
Good catch sorry. I'll fix this and add a test for it
Sorry, something went wrong.
Fixed in 1.11.1
No branches or pull requests
Hi @JanEbbing
I saw you introduced a new DeeplClient classes which is supposed to be used instead of Translator.
cf 46b68a7
There is also a DeeplClientOptions class, but I don't see why it extends
TranslateTextOptions
.Shouldn't it extends
TranslatorOptions
instead ?In the same way I did
i would expect now to do
WDYT ?
The text was updated successfully, but these errors were encountered: