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

Translation w/ glossary entries are only partially translated and include sourceLang string #20

Open
AltanS opened this issue Feb 28, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@AltanS
Copy link

AltanS commented Feb 28, 2023

Describe the bug
After creating a glossary and running a test translation, the glossary entry provided was partially translated and the source entry is sometimes included in the translation.

To Reproduce
Steps to reproduce the behavior:

  1. I have created a glossary from EN to DE
  2. I have added some entries to test the functionality
  3. Results as seen in the screenshot below are not sticking to the terms set in the glossary.

image

This is the code I'm calling the translations with:

  const testTranslation3 = await translator.translateText('test1', 'en', 'de', {
    glossary: glossaryId,
  })

Expected behavior
I'm uncertain if I misunderstood how glossaries work, but I thought they would always take precedence over default translations. Also, the source language string should never show up in the translation.

@seekuehe seekuehe added the bug Something isn't working label Apr 17, 2023
@STR1234
Copy link

STR1234 commented Apr 24, 2024

I have experienced the same behaviour and just tried with a normal fetch with the same test glossary_id of a glossary I created. It seems to be related to the package as the vanilla fetch call behaves as expected with the glossary.
As a workaround for now the vanilla fetch call will have to suffice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants