Skip to content

Add pre-translate option to Phrase plugin - #4820

Open
sanyamkamat wants to merge 2 commits into
mainfrom
ai_main_0430da72489e490f99eb
Open

Add pre-translate option to Phrase plugin#4820
sanyamkamat wants to merge 2 commits into
mainfrom
ai_main_0430da72489e490f99eb

Conversation

@sanyamkamat

@sanyamkamat sanyamkamat commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Description

Added a "Pre-translate" checkbox option to the Phrase plugin's job creation dialog, allowing users to opt in to using approved translations from Phrase Translation Memory to pre-fill a job.

Changes include:

  • Added a preTranslate field (defaulting to false) to the configuration editor's local store.
  • Added a FormControlLabel with a Checkbox UI control in snackbar-utils.tsx to let users toggle the pre-translate option, with a description explaining its behavior.
  • Updated the getLangPicks return type and onChoose callback to include preTranslate.
  • Passed the preTranslate value through plugin.tsx when creating a job.
  • Updated PhraseApi.createJob to accept and send the preTranslate flag in the job creation request body.

Link to JIRA ticket (if applicable):
https://builder-io.atlassian.net/browse/ENG-13444


Edit in Builder  Preview


To clone this PR locally use the Github CLI with command gh pr checkout 4820

You can tag me at @BuilderIO for anything you want me to fix or change


Note

Low Risk
Scoped plugin UI and job-creation payload with default false; behavior unchanged unless users opt in, though the backend must accept the new field.

Overview
Adds an optional Pre-translate control to the Phrase connector’s translate dialog so editors can request Translation Memory pre-fill when starting a job.

The language picker dialog now includes a checkbox (default off) with copy explaining that approved TM matches can pre-fill the job. That choice is carried through getLangPickscreateJob as a preTranslate boolean on POST /api/v1/memsource/job. Package version is bumped to 0.0.18-0.

Reviewed by Cursor Bugbot for commit 17562b1. Bugbot is set up for automated code reviews on this repo. Configure here.

@changeset-bot

changeset-bot Bot commented Aug 21, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 17562b1

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@builder-io-integration builder-io-integration Bot changed the title Update from the Builder.io agent Add pre-translate option to Phrase plugin Aug 21, 2026
@sanyamkamat
sanyamkamat requested review from a team and AishwaryaParab and removed request for a team August 21, 2026 06:20
@nx-cloud

nx-cloud Bot commented Aug 21, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 17562b1

Command Status Duration Result
nx test @snippet/react ✅ Succeeded 58s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-09-10 14:58:13 UTC

builder-io-integration[bot]

This comment was marked as outdated.

@builder-io-integration builder-io-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Builder reviewed your changes — looks good ✅

Review Details

Incremental Code Review Summary

The latest PR update adds the 0.0.18-0 package version and synchronizes the corresponding package-lock entries. The existing pre-translate implementation remains internally consistent: the checkbox defaults off, its value is returned by the dialog, passed through the plugin, and included in the Phrase job payload.

Risk assessment: Standard. This is a scoped plugin UI/API integration change with a version bump and no authentication, billing, destructive, or schema changes.

Two independent incremental review passes found no genuinely new confirmed issues. The package metadata and lockfile versions match, and no regressions were identified in the translation flow. No comments are being submitted.

🧪 Browser testing: Skipped — dev server unavailable on HEAD while dependency setup is still failing/blocked during yarn install (including TypeScript patch errors); will retry on the next incremental review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants