fix: source UT PDF list titles from locale files instead of hardcoded strings - #847
Conversation
… strings The createUtDailyHearingListPdfGenerator factory previously accepted courtName and listTitle as hardcoded English strings, meaning Welsh PDFs would always display English court names. Translations are now loaded first and t.pageTitle is passed as listTitle so the correct locale string is used. The unused courtName parameter is removed throughout. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 35 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (11)
📝 WalkthroughWalkthroughThe shared Upper Tribunal PDF generator now loads translations before rendering and supplies ChangesUpper Tribunal PDF title localisation
Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ 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 |
🎭 Playwright E2E Test Results84 tests 52 ✅ 7m 55s ⏱️ Results for commit f4d049f. ♻️ This comment has been updated with latest results. |
…when no E2E results When this PR has no E2E test changes, dorny/test-reporter errors because junit-results.xml doesn't exist. fail-on-empty: false makes it a no-op instead of a failure. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
courtName was removed from RenderOptions in all three UT renderer interfaces; the web page controllers still passed it, causing TS2353 build errors. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…b controllers Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Preview Deployment Successful 🚀Your preview environment is ready:
The environment will be automatically cleaned up when this PR is closed. |



Summary
createUtDailyHearingListPdfGeneratorpreviously acceptedcourtNameandlistTitleas hardcoded English strings, so Welsh PDFs always displayed English court/list namest.pageTitle(from the locale file) is passed aslistTitleso the correct locale string renders in the PDFcourtNameparameter is removed from the factory signature and all three rendererRenderOptionsinterfacesAffected packages
@hmcts/upper-tribunal-common(factory)@hmcts/upper-tribunal-administrative-appeals-chamber-daily-hearing-list@hmcts/upper-tribunal-lands-chamber-daily-hearing-list@hmcts/upper-tribunal-tax-and-chancery-chamber-daily-hearing-listTest plan
listTitlenow reads fromcy.pageTitlee.g."Rhestr Gwrandawiadau Dyddiol Tribiwnlys Uwch (Siambr Apeliadau Gweinyddol)"listTitlereads fromen.pageTitleas before🤖 Generated with Claude Code
Summary by CodeRabbit
Enhancements
Bug Fixes