Skip to content

fix(i18n): translate AI image drop hint - #790

Open
mardausdennis wants to merge 1 commit into
frappe:developfrom
mardausdennis:fix/translate-ai-drop-hint
Open

fix(i18n): translate AI image drop hint#790
mardausdennis wants to merge 1 commit into
frappe:developfrom
mardausdennis:fix/translate-ai-drop-hint

Conversation

@mardausdennis

@mardausdennis mardausdennis commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

The Drop image to attach string was previously present in the POT, but the AI panel rewrite lost its translation wrapper. This restores the existing __() pattern for that hint.

@greptile-apps

greptile-apps Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

The PR appears safe to merge with no actionable issues identified.

The translation helper is globally registered, typed, and already used through the same template pattern throughout the frontend.

Reviews (1): Last reviewed commit: "fix(i18n): translate AI image drop hint" | Re-trigger Greptile

@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.06%. Comparing base (12c787a) to head (237a775).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #790   +/-   ##
========================================
  Coverage    66.06%   66.06%           
========================================
  Files           93       93           
  Lines        10386    10386           
========================================
  Hits          6862     6862           
  Misses        3524     3524           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant