Skip to content

Fix getGPTCommand method to include targetLocale in extend call#11

Open
Zazama wants to merge 2 commits into
wernerkrauss:mainfrom
Zazama:main
Open

Fix getGPTCommand method to include targetLocale in extend call#11
Zazama wants to merge 2 commits into
wernerkrauss:mainfrom
Zazama:main

Conversation

@Zazama
Copy link
Copy Markdown
Contributor

@Zazama Zazama commented Sep 23, 2025

The README.md has a passage about extending the prompt dynamically, but the $locale parameter is not passed to the extension call.

public function updateGptCommand(&$command, $locale)
{
    $command = 'You are a professional translator. Translate the following text to ' . $locale . ' language. Please keep the json format intact.';
}

@adiwidjaja
Copy link
Copy Markdown
Contributor

We moved the autotranslate functionality here: https://github.com/s2hub/silverstripe-autotranslate and included this functionality.

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