Skip to content

Fix hardcoded string composition with internationalizations (#5293)#5312

Open
bstepanovski wants to merge 5 commits intomasterfrom
bug-fix/hardcoded-translation-strings-5293
Open

Fix hardcoded string composition with internationalizations (#5293)#5312
bstepanovski wants to merge 5 commits intomasterfrom
bug-fix/hardcoded-translation-strings-5293

Conversation

@bstepanovski
Copy link
Copy Markdown
Contributor

Fixes #5293

  • Replaced hardcoded string compositions where internationalized strings are used to avoid antipatterns
  • Added missing translations in all locales and updated tests accordingly
  • No behavior changes, just cleanup to make translations work correctly across languages

Comment thread spec/e2e/nginx_stage_spec.rb Outdated
Comment on lines +10 to +15
before(:all) do
upload_portal_config('portal.yml')
update_ood_portal
restart_apache
restart_dex
end
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm hesitant to add this. Seems like it could cause more flakiness to be bouncing things all the time.

Copy link
Copy Markdown
Contributor

@Bubballoo3 Bubballoo3 left a comment

Choose a reason for hiding this comment

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

It seems like the only open question is the modifications to the E2E tests, so you should just split that out into a separate change so we can properly discuss them

@github-project-automation github-project-automation Bot moved this from Awaiting Review to Changes Requested in PR Review Pipeline Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Changes Requested

Development

Successfully merging this pull request may close these issues.

Don't compose strings with internationalizations

4 participants