Skip to content

fix: Update German translation for push sharing message - #4682

Open
GoliathLabs wants to merge 1 commit into
pglombardo:masterfrom
GoliathLabs:patch-1
Open

fix: Update German translation for push sharing message#4682
GoliathLabs wants to merge 1 commit into
pglombardo:masterfrom
GoliathLabs:patch-1

Conversation

@GoliathLabs

@GoliathLabs GoliathLabs commented Jul 28, 2026

Copy link
Copy Markdown

Description

Apparently, this was auto translated. "Pressemitteilung" doesn't make sense here.

Related Issue

Type of Change

  • 📦 Dependency & security updates
  • 🔧 Bug fix (non-breaking change which fixes an issue)
  • 🥂 Improvement (non-breaking change which improves an existing feature)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🔐 Security fix
  • 📚 Examples / documentation / tutorials

Checklist

  • I've written tests (if applicable) for all new methods and classes that I created. (rake test)
  • I've added documentation as necessary so users can easily use and understand this feature/fix.

Note

Low Risk
Single German gettext string correction with no runtime or security impact.

Overview
Fixes the German string on the push preview page (preview.html.erb) for "Copy the link or message below to share this push securely."

The previous translation used "Pressemitteilung" (press release), which is a bad auto-translation for Password Pusher’s push feature. It now uses "diesen Push", consistent with other German entries in the same locale file (e.g. push created / push ownership strings).

Only config/locales/gettext/de/app.po changes; no application logic.

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

Signed-off-by: Felix <8057646+GoliathLabs@users.noreply.github.com>
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