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

Use 'gtx' instead of 't' as client parameter. #99

Merged
merged 1 commit into from
Jul 22, 2020

Conversation

kdmsnr
Copy link
Contributor

@kdmsnr kdmsnr commented Dec 4, 2019

Use 'gtx' instead of 't' as client parameter.
(see matheuss/google-translate-api#79 )

Use 'gtx' instead of 't' as ``client`` parameter.
(see matheuss/google-translate-api#79 )
@kdmsnr
Copy link
Contributor Author

kdmsnr commented Dec 4, 2019

Sorry, it seems the same PR as #95.

@kdmsnr kdmsnr closed this Jun 17, 2020
@kdmsnr kdmsnr deleted the patch-1 branch June 17, 2020 12:53
@kdmsnr kdmsnr restored the patch-1 branch June 25, 2020 16:15
@kdmsnr
Copy link
Contributor Author

kdmsnr commented Jun 25, 2020

I think it still needs this patch.

@stardiviner
Copy link
Collaborator

stardiviner commented Jul 22, 2020

Any clue about the difference with #95 ?

@kdmsnr
Copy link
Contributor Author

kdmsnr commented Jul 22, 2020

@stardiviner
Copy link
Collaborator

stardiviner commented Jul 22, 2020

I can't reproduce this problem on my computer. But I checked your referenced links and this issue background. Many users solved by this. And I also tested your "client": "gtx" patch, it works on my machine too. So I think this patch should be merged. @kdmsnr Thanks for your PR!

@stardiviner stardiviner merged commit 619d318 into atykhonov:master Jul 22, 2020
@kdmsnr kdmsnr deleted the patch-1 branch July 22, 2020 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants