Skip to content

DEV: Remove automatic translations #310

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

Merged
merged 4 commits into from
May 29, 2025
Merged

DEV: Remove automatic translations #310

merged 4 commits into from
May 29, 2025

Conversation

nattsw
Copy link
Contributor

@nattsw nattsw commented May 28, 2025

Requires to be merged: #308

Removes automatic translations and the DiscourseAI provider.

This also includes a migration that disables the plugin if DiscourseAI is selected.

Base automatically changed from remove-anon-switcher to main May 29, 2025 03:37
@nattsw nattsw force-pushed the trans-job branch 2 times, most recently from 02eec6a to 9aac624 Compare May 29, 2025 09:07
@nattsw nattsw merged commit 0ca435a into main May 29, 2025
12 checks passed
@nattsw nattsw deleted the trans-job branch May 29, 2025 09:12
nattsw added a commit to discourse/discourse-ai that referenced this pull request May 29, 2025
…egories (#1376)

Related: discourse/discourse-translator#310

This commit includes all the jobs and event hooks to localize posts, topics, and categories.

A few notes:
- `feature_name: "translation"` because the site setting is `ai-translation` and module is `Translation`
- we will switch to proper ai-feature in the near future, and can consider using the persona_user as `localization.localizer_user_id`
- keeping things flat within the module for now as we will be moving to ai-feature soon and have to rearrange
- Settings renamed/introduced are:
  - ai_translation_backfill_rate (0)
  - ai_translation_backfill_limit_to_public_content (true)
  - ai_translation_backfill_max_age_days (5)
  - ai_translation_verbose_logs (false)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants