Skip to content

feat(Authoring): Translate with AI#314

Merged
hirokiterashima merged 14 commits intodevelopfrom
translate-with-ai
Apr 12, 2026
Merged

feat(Authoring): Translate with AI#314
hirokiterashima merged 14 commits intodevelopfrom
translate-with-ai

Conversation

@Aaron-Detre
Copy link
Copy Markdown
Contributor

@Aaron-Detre Aaron-Detre commented Feb 22, 2026 •
edited
Loading

Changes

  • Added an API endpoint to handle AWS Translate requests.
  • Changed Controller -> RestController since saveTranslations already had ResponseBody.

Testing

  • Test with feat(Authoring): Translate with AI WISE-Client#2272.
  • Add the properties below to application-dockerdev.properties.
    • If you don't set values for these, the translate with AI button should not appear.
    • If the values you set don't work, clicking the translate with AI button should give you an alert with a warning.
aws.accessKeyId=
aws.secretAccessKey=
aws.region=

@Aaron-Detre Aaron-Detre self-assigned this Feb 22, 2026
@Aaron-Detre Aaron-Detre added the enhancement New feature or request label Feb 22, 2026
@Aaron-Detre Aaron-Detre changed the title feat(Authoring Tool): Translate with AI feat(Authoring): Translate with AI Feb 22, 2026
@Aaron-Detre Aaron-Detre mentioned this pull request Feb 23, 2026
Merged
@Aaron-Detre Aaron-Detre marked this pull request as ready for review February 23, 2026 04:28
hirokiterashima
hirokiterashima requested changes Mar 3, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@hirokiterashima hirokiterashima left a comment

Choose a reason for hiding this comment

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

  • We need to handle case when aws settings are not specified in application.properties. Right now, the API throws an exception and won't start.
  • Add the new aws settings to application_sample.properties and application-dockerdev-sample.properties, with explanation on what they're used for.

@hirokiterashima hirokiterashima requested a review from breity March 24, 2026 15:54
@hirokiterashima
Merge branch 'develop' into translate-with-ai
a501bd8 
~ Conflicts:
~	src/main/java/org/wise/portal/presentation/web/controllers/author/project/AuthorAPIController.java
~	src/test/java/org/wise/portal/presentation/web/controllers/author/project/AuthorAPIControllerTest.java
hirokiterashima
hirokiterashima approved these changes Apr 12, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@hirokiterashima hirokiterashima left a comment

Choose a reason for hiding this comment

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

LGTM

@hirokiterashima hirokiterashima merged commit e86ed66 into develop Apr 12, 2026
2 checks passed
@hirokiterashima hirokiterashima deleted the translate-with-ai branch April 12, 2026 05:39
@hirokiterashima
Copy link
Copy Markdown
Member

hirokiterashima commented Apr 12, 2026

🎉 This PR is included in version 1.19.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Reviewers

@hirokiterashima hirokiterashima hirokiterashima approved these changes

@breity breity Awaiting requested review from breity

Assignees

@Aaron-Detre Aaron-Detre

Labels

enhancement New feature or request released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Aaron-Detre @hirokiterashima