Skip to content
Snippets Groups Projects

DeepLX-JS support

Merged Marie requested to merge feature/deeplx into develop
All threads resolved!

What does this PR do?

This allows instance admins to use DeepLX as an alternative to DeepL for translations

There is already a doc MR from before the move:

Documentation!14 (merged)

Some Information can be found on the issue discussing alternative translation options:

#324 (closed)

Contribution Guidelines By submitting this merge request, you agree to follow our Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
  • I have made sure to test this pull request

Additional Note i18n will be added to this after I am back from my break considering this code was already done before the move to GitLab and co.

I just felt like opening this MR already so people can look a it.

Edited by Marie

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • dakkar
    • Developer
      Resolved by dakkar

      I still think that DeepLX-JS is a nasty hack, but this patch is small enough to accept (once that weirdness with the URL prefix gets explained)

  • Marie added 1 commit

    added 1 commit

    Compare with previous version

  • Ghost User approved this merge request

    approved this merge request

  • dakkar resolved all threads

    resolved all threads

  • dakkar approved this merge request

    approved this merge request

  • Marie added 1 commit

    added 1 commit

    Compare with previous version

  • Marie added 1 commit

    added 1 commit

    Compare with previous version

  • merged

  • Marie mentioned in commit d3d0e510

    mentioned in commit d3d0e510

  • Please register or sign in to reply
    Loading