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

little bug using Bing translator #546

Closed
edwardyeung opened this issue Dec 23, 2024 · 9 comments
Closed

little bug using Bing translator #546

edwardyeung opened this issue Dec 23, 2024 · 9 comments
Assignees
Labels
bug Something isn't working fixed Fixed in develop or master, to be launched.

Comments

@edwardyeung
Copy link

Description

I was trying to add Bing translator.

To Reproduce

Steps to reproduce the behavior, e.g.:

  1. Go to 'Language'
  2. add new dictionary, choose embedded, and click the little arrow to test, you will find everything is normal
  3. Go to book, and click any words to translate via Bing, then the error shows up.

Error logs

Version: 3.7.0
You were on: http://192.168.50.215:5001/read/17
You were trying to access: http://192.168.50.215:5001/bing/search/4/find/https://www.bing.com/translator%3Ftext=[LUTE]
image

Other
installed in OrangePi 3 LTS. Python 3.10

@jzohrab
Copy link
Collaborator

jzohrab commented Dec 23, 2024

Hi @edwardyeung , thanks for the message. Can you paste the URL that you set up in the dictionary list for the language? A screenshot is fine. Cheers!

@jzohrab jzohrab self-assigned this Dec 23, 2024
@jzohrab jzohrab added this to Lute-v3 Dec 23, 2024
@jzohrab jzohrab moved this to In Progress in Lute-v3 Dec 23, 2024
@jzohrab jzohrab added the question Further information is requested label Dec 23, 2024
@edwardyeung
Copy link
Author

url:
https://www.bing.com/translator?text=[LUTE]

I thought the problem caused by the question mark at first, so I tried "https://www.bing.com/translator", and had no luck.

@jzohrab
Copy link
Collaborator

jzohrab commented Dec 23, 2024

I think you need the from and to parameters as well. For example, here is from Thai to English:

https://www.bing.com/translator/?from=th&to=en&text=[LUTE]

@edwardyeung
Copy link
Author

by the way, I don't know if you are interested or not.
I am an obsidian (a note taking app) user, there is a plugin for obsidian called custom frame which solves the problems of some websites forbidding using iframe and make users access google keep, google translate...etc

here is the plugin source.
https://github.com/Ellpeck/ObsidianCustomFrames/blob/master/src/frame.ts

@edwardyeung
Copy link
Author

I think you need the from and to parameters as well. For example, here is from Thai to English:

https://www.bing.com/translator/?from=th&to=en&text=[LUTE]

Brilliant!

@edwardyeung
Copy link
Author

I think you need the from and to parameters as well. For example, here is from Thai to English:

https://www.bing.com/translator/?from=th&to=en&text=[LUTE]

sadly, still had no luck using the same url.
image

@jzohrab
Copy link
Collaborator

jzohrab commented Dec 23, 2024

Well, this is annoying! Back at my computer, trying further on my machine, and I have the same issue :-( . It used to work, but I couldn't test it in CI so ... broken silently, which sucks terribly. Thanks for the ping, there are some things that just can't be tested easily unless it's done manually (unless, of course, I just don't know what I'm missing).

Anyway, reproducing: For Spanish, I have the following as an embedded term dict:

https://www.bing.com/translator/?from=es&to=en&text=[LUTE]

i.e.

image

and I got this error:

image

Thank you @edwardyeung , I'll try to get a fix out very soon, as it's nice to have the embedded bing lookup. Cheers!

@jzohrab jzohrab added bug Something isn't working and removed question Further information is requested labels Dec 23, 2024
@jzohrab jzohrab moved this from In Progress to Todo in Lute-v3 Dec 23, 2024
@jzohrab jzohrab moved this from Todo to In Progress in Lute-v3 Dec 23, 2024
@jzohrab
Copy link
Collaborator

jzohrab commented Dec 23, 2024

OK, this is fixed in develop now. Here's using Bing as a term lookup:

Image

and as a sentence lookup:

Image

@jzohrab jzohrab moved this from In Progress to Done in Lute-v3 Dec 23, 2024
@jzohrab jzohrab added the fixed Fixed in develop or master, to be launched. label Dec 25, 2024
@jzohrab
Copy link
Collaborator

jzohrab commented Dec 25, 2024

Launched in 3.8.0.

@jzohrab jzohrab closed this as completed Dec 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed Fixed in develop or master, to be launched.
Projects
Archived in project
Development

No branches or pull requests

2 participants