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

feat: support DeepL Free API #2

Merged
merged 2 commits into from Feb 7, 2022

Conversation

thebuilder
Copy link
Contributor

Add support for the DeepL Free API tier. It needs to target another endpoint, but otherwise it functions the same.
This allows you to try out the DeepL service, and switch to the paid plan by switching out the endpoint.

Fixes #1

@netlify
Copy link

netlify bot commented Feb 1, 2022

✔️ Deploy Preview for datocms-plugin-translate-fields ready!

🔨 Explore the source changes: c1c45e1

🔍 Inspect the deploy log: https://app.netlify.com/sites/datocms-plugin-translate-fields/deploys/61ff0f1172521d0008cc9c42

😎 Browse the preview: https://deploy-preview-2--datocms-plugin-translate-fields.netlify.app/

Copy link
Member

@vvandijck vvandijck left a comment

Choose a reason for hiding this comment

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

@thebuilder Very nice addition to this DatoCMS plugin. 💅🏼 I have 2 small suggestions.

@velomovies what do you think?

src/lib/translation.ts Outdated Show resolved Hide resolved
src/lib/translation.ts Outdated Show resolved Hide resolved
Add suggestions
Copy link
Contributor

@velomovies velomovies left a comment

Choose a reason for hiding this comment

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

Thanks for the request.

Nice addition 👍

@velomovies velomovies merged commit 05f1583 into voorhoede:main Feb 7, 2022
@thebuilder thebuilder deleted the feat/deepl-free branch February 7, 2022 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support DeepL free API
3 participants