Skip to content

Fix views, language key collisions, and minor UI issues - #33

Open
jpmorby wants to merge 1 commit into
blesta:masterfrom
jpmorby:pr/opensrs-views-language-fixes
Open

jpmorby wants to merge 1 commit into
blesta:masterfrom
jpmorby:pr/opensrs-views-language-fixes

Conversation

@jpmorby

@jpmorby jpmorby commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fixes duplicate/colliding language keys across the module's view templates
  • Fixes minor UI issues in the admin/client view templates

Test plan

  • Load each affected admin and client service tab and confirm labels render correctly with no missing/incorrect text
  • Confirm no PHP notices/warnings are raised by the language file when views are rendered

- Mask API key in manage view, showing only last 4 characters
- Change API key input to a password field in add/edit row views
- Remove unused $i variable in client WHOIS tab view
- Namespace .FR domain language keys (fr_registrant_type) to prevent
  collision with .UK registrant_type keys that share the same key
  name; also fixes a pre-existing bug where .FR referenced a
  nonexistent Opensrs.domain.registrant_type.company language key
- Update .FR config to reference the namespaced language keys
@pmphillips

Copy link
Copy Markdown
Member

Code review passed — the .FR/.UK language-key separation, removal of the nonexistent registrant_type.company reference, and the corrected password-field signature all check out. No findings. Merge pending final review and manual testing under CORE-6158.

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.

2 participants