Loop DeepL suggestions through the backend

matthiask commented 2 years ago

DeepL wants an Authorization header but we cannot set this in the JavaScript code because of missing CORS headers on the DeepL side.

Opening as a draft pull request because this change introduces a different way to fetch suggestions from third party services.

[x] Are tests passing? (From the root-level of the repository please run pip install tox && tox)
[ ] I have added or updated a test to cover the changes proposed in this Pull Request
[ ] I have updated the documentation to cover the changes proposed in this Pull Request

matthiask commented 2 years ago

Ah... I see that Google and Azure are handled the same way already. Let me clean this up.

mbi commented 1 year ago

Hello and thank you so much for your PR. I just noticed it, right after pushing my own correction for #271, sorry about that. :facepalm:

Maybe you'd like to take a look at my implementation, and if if you notice anything worth improving, please do not hesitate!

matthiask commented 1 year ago

Thanks for the fix! LGTM.

mbi / django-rosetta