Skip to content

fix: OpenAI max_tokens deprecated & support API retry on request failure#72

Merged
Siilwyn merged 4 commits into
voorhoede:mainfrom
elibolonur:main
Nov 14, 2025
Merged

fix: OpenAI max_tokens deprecated & support API retry on request failure#72
Siilwyn merged 4 commits into
voorhoede:mainfrom
elibolonur:main

Conversation

@elibolonur

Copy link
Copy Markdown
Contributor

Changes

  • Fixed: OpenAI API deprecation: replaced max_tokens with max_completion_tokens and updated label in the UI
  • Added: Retry button for failed API calls, allowing users to quickly re-run translation requests

Technical Details

  • Updated OpenAI request builder to use max_completion_tokens
  • Improved error handling to surface clear messages and provide retry option

Testing

  • Verified translation requests succeed with gpt-5-mini using max_completion_tokens
  • Confirmed Retry button replays failed requests successfully
  • All existing tests passing

@netlify

netlify Bot commented Sep 16, 2025

Copy link
Copy Markdown

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

Name Link
🔨 Latest commit 6332f45
🔍 Latest deploy log https://app.netlify.com/projects/datocms-plugin-translate-fields/deploys/6917597aaf83610008d50fd6
😎 Deploy Preview https://deploy-preview-72--datocms-plugin-translate-fields.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Siilwyn

Siilwyn commented Nov 14, 2025

Copy link
Copy Markdown
Member

Thank you for your contribution. I've had to update the branch to run the test workflow, all green!

@Siilwyn Siilwyn merged commit b823d1d into voorhoede:main Nov 14, 2025
5 checks passed
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.

2 participants