Hello and thanks for this nice module:)

Problem/Motivation

Do you have plans for LibreTranslate integration?

Best,
Orkut

CommentFileSizeAuthor
#8 3399617.patch11.44 KBjmauricio
#6 3399617.patch5.12 KBjmauricio

Comments

Orkut Murat Yılmaz created an issue. See original summary.

orkutmuratyilmaz’s picture

I've also just found the LibreTranslate module. It would be nice to merge the efforts:)

orkutmuratyilmaz’s picture

kallado’s picture

Not really but if you are interessed in helping out you can take a look at how the amazon API was implementede here
https://www.drupal.org/project/auto_node_translate/issues/3322954
We are kind of busy right now and still have some opened issues (bugs) to fix but we can take a look wenever we have some spare minutes.

orkutmuratyilmaz’s picture

@kallado, thanks for this beautiful comment:)

I'm waiting for @dcimorra's comment now. If they don't want to do, I'll give it a try on this issue, next month.

jmauricio’s picture

Assigned: Unassigned » jmauricio
Status: Active » Needs work
StatusFileSize
new5.12 KB

@Orkut Murat Yılmaz or anyone else interested.

Can you test this patch, it integrates LibreTranslate into this module.
I've tested with a self-hosted version, works fine.

Thanks

jmauricio’s picture

Version: 2.1.7 » 2.1.9
Assigned: jmauricio » Unassigned
Status: Needs work » Needs review
jmauricio’s picture

StatusFileSize
new11.44 KB

The patch didn't include the api file, here's the correct one

orkutmuratyilmaz’s picture

@jmauricio thanks for your patch. I've just checked the code and it looks fine. I'm going to test it in a day or two.

kallado’s picture

Tested and it looks fine

  • jmauricio committed 5b7210fe on 1.0.x
    Issue #3399617 by jmauricio, Orkut Murat Yılmaz, kallado: LibreTranslate...
jmauricio’s picture

Status: Needs review » Reviewed & tested by the community
jmauricio’s picture

Status: Reviewed & tested by the community » Fixed
jmauricio’s picture

Status: Fixed » Closed (fixed)
orkutmuratyilmaz’s picture

Thanks for the merge with the stable branch:)

Can you add this one to the module description too?