fix(serbian): drop invented {…} placeholder from 'Invalid email address or placeholder detected' - #70
Conversation
…ected'
Original Serbian translation introduced a {…} format-token into an
entry where the English source ('Invalid email address or placeholder
detected') has no placeholder. Rewriting as a more literal mapping
('Neispravna email adresa ili čuvar mesta.') matches the source
structure without inventing a UI token.
Flagged in the anti-slop pass requested in -5292199939 by @teamssUTXO
2026-05-26. Sanya (@Sanja22B) - maintainer of serbian.json - asked
for a PR with this fix.
I'm not a native Serbian speaker; the choice of 'čuvar mesta' for
'placeholder' is a literal best-attempt mapping. If a different word
or phrasing fits better in Serbian software conventions, please push
a maintainer-edit on this branch or comment with the preferred
wording and I'll update.
The broader identical-to-English entries from the same pass (Server,
Token, Limit, Port, Checkout, Crowdfund, Placeholders, Plugin server,
Value Mapper, etc.) are judgment calls on which tech anglicisms to
keep vs translate, deliberately left out of this PR for the
maintainer to decide. Full report markdown available on request.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Serbian translation file is updated with a shorter translation for an email validation error message key. The new wording provides a more concise version of the message while maintaining the meaning. ChangesTranslation Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Closing - @Sanja22B explained in |
2 participants
Per @Sanja22B's request in
-52921999392026-05-26 13:34 UTC, opening this PR from the anti-slop pass on serbian.json.What
One entry in
translations/serbian.jsonintroduced a{…}format-token in the Serbian translation that doesn't exist in the English source:Invalid email address or placeholder detectedJedna od email adresa ili vrednosti u formatu {…} za primaoca nije ispravna.Neispravna email adresa ili čuvar mesta.The
{…}ellipsis-in-braces was the translator's attempt to render "placeholder" concretely by showing what one looks like. It works, but it doesn't match the source structure and other localizations don't use this device.Rewriting as a more literal mapping aligns with the source.
Caveats
I'm not a native Serbian speaker.
čuvar mestais a literal best-attempt translation of "placeholder" ("place-keeper"). If a different word or phrasing fits Serbian software-localization conventions better, please push a maintainer-edit on this branch or comment with the preferred wording and I'll update.Out of scope
The anti-slop pass also flagged 38 other entries in serbian.json as identical-to-English (Server, Token, Limit, Port, Checkout, Crowdfund, Placeholders, Plugin server, Value Mapper, etc.). These are judgment calls on which tech anglicisms to keep vs translate, deliberately left for the maintainer to decide rather than batched into this fix. Full report markdown available on request.
Context
Anti-slop pass triggered by @teamssUTXO msg 472 / 476 / 478 in the BTCPay Translations Telegram group 2026-05-26. Pass scope:
_maintainer-tagged languages except French + Spanish (Portuguese-BR, Serbian, Hindi). Sanya is the serbian.json maintainer per_maintainer: sanya|https://github.com/Sanja22B.