Skip to content

Bug: Missing translation when authentication fails#2480

Merged
mroderick merged 1 commit intocodebar:masterfrom
mroderick:fix-missing-authentication-error-translation
Feb 11, 2026
Merged

Bug: Missing translation when authentication fails#2480
mroderick merged 1 commit intocodebar:masterfrom
mroderick:fix-missing-authentication-error-translation

Conversation

@mroderick
Copy link
Collaborator

Description

Fixes #2474

This PR adds the missing translation string notifications.authentication_error for all available languages.

Changes

  • Added authentication_error translation key to all locale files:
    • English variants (en, en_AU, en_GB, en_US)
    • German (de)
    • Spanish (es)
    • Finnish (fi)
    • French (fr)
    • Norwegian (no)

Problem

When authentication fails, the application was displaying:

Translation missing: en.notifications.authentication_error

Solution

Added the appropriate translation strings so users now see:

Authentication failed. Please try again.

(or the equivalent message in their selected language)

Testing

The translation key is used in app/controllers/auth_services_controller.rb:63 in the failure action when OAuth authentication fails.

Adds the missing translation string 'notifications.authentication_error'
for all available languages (en, en_AU, en_GB, en_US, de, es, fi, fr, no).

This resolves the "Translation missing: en.notifications.authentication_error"
error that was displayed when authentication fails.

Fixes codebar#2474
Copy link
Contributor

@KimberleyCook KimberleyCook left a comment

Choose a reason for hiding this comment

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

🎉

@mroderick mroderick merged commit 5c9c428 into codebar:master Feb 11, 2026
14 of 15 checks passed
@mroderick mroderick deleted the fix-missing-authentication-error-translation branch February 11, 2026 12:01
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.

Bug: Missing translation when authentication fails

2 participants