Skip to content
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

Update URL and client #138

Merged
merged 2 commits into from
Nov 28, 2022
Merged

Update URL and client #138

merged 2 commits into from
Nov 28, 2022

Conversation

Razzo1987
Copy link
Contributor

For me stop working without this fix

@Stichoza
Copy link
Owner

Strange, what error were you getting? Just tested and everything works fine for me.

@metikulos

This comment has been minimized.

@taiviemthoi
Copy link

Hi @Stichoza
#23

<div class='love' id='green'>i love you<div><img src='https://exem.com' alt=''>

en->fr

<div class = 'love' id = "vert"> je t'aime <div> <: //exem.com' alt = ''>

id html is translated, or with other language (eg -> vi) it automatically capitalizes the css class

img src url with extra space

But when I try with client=gtx and the endpoint is https://translate.googleapis.com/translate_a/single
then the translation is more accurate

<div class='love' id='green'>je t'aime<div><img src='https://exem.com' alt=''>

so could you please merge this request?

Copy link

@taiviemthoi taiviemthoi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sunxiyuan
Copy link

I encountered the same problem and successfully solved it by changing the client to gtx

@sunxiyuan
Copy link

To add, when the content is as follows, if the client is not changed to gtx, an exception will be returned

If a new user is created by WordPress, a new password will be randomly generated and the new user&#8217;s role will be set as %s. Manually changing the new user&#8217;s details will be necessary.

@Stichoza Stichoza merged commit c931540 into Stichoza:master Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants