Skip to content
This repository has been archived by the owner on Sep 7, 2023. It is now read-only.

Deepl #3332

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Deepl #3332

wants to merge 2 commits into from

Conversation

cy8aer
Copy link
Contributor

@cy8aer cy8aer commented Aug 8, 2022

What does this PR do?

This implements the Deepl Translation engine

It works nearly like lingva but directly to the deepl api. There is a free option to use. See https://www.deepl.com/pro-api?cta=header-pro-api for registering a free account.

This api only needs a to-lang, from-lang is a fake by now

Why is this change important?

You will have deepl in searx with full phrase and sentence translations

How to test this PR locally?

Api key needed. Put api key into settings.yml - enable deepl.

de-en Dies ist ein großartiger Test
en-es This is deepl.

Author's checklist

Related issues

@@ -1401,6 +1401,15 @@ engines:
# See : https://mymemory.translated.net/doc/usagelimits.php
# api_key : ''

- name : deepl
Copy link
Member

Choose a reason for hiding this comment

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

As the engine cannot function without an API key, please comment it out.

@cy8aer
Copy link
Contributor Author

cy8aer commented Aug 9, 2022

Uh, I hope I did the merging right. I need some routine for doing this.
For the patch: It is now commented out.

@cy8aer
Copy link
Contributor Author

cy8aer commented Aug 9, 2022

Removed update branch from upstream.

@cy8aer
Copy link
Contributor Author

cy8aer commented Aug 9, 2022

and rebased it to master. This should be the right way.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants