Skip to content

fix: localize gender labels in admin user details - #371

Closed
fabiodalez-dev wants to merge 2 commits into
mainfrom
fix-gender-display-admin-user-details
Closed

fix: localize gender labels in admin user details#371
fabiodalez-dev wants to merge 2 commits into
mainfrom
fix-gender-display-admin-user-details

Conversation

@fabiodalez-dev

Copy link
Copy Markdown
Owner

Supersedes #369 with the original contributor commit preserved, plus the review fixes required to make the change merge-ready. The external fork has maintainer edits disabled, so its branch could not be updated directly.

Changes:

  • render stored M, F, and Altro values through the locale catalogue;
  • preserve legacy decode-then-escape behavior for all existing detail fields;
  • add a stable, locale-independent field hook;
  • cover real registration persistence and all enum labels in the serial issue Configurable registration fields #255 browser suite.

Validation:

Co-authored work from @Himura2la is preserved in the branch history.

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@fabiodalez-dev, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 5 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 83ad5030-a16c-41f4-a589-079ee3034a80

📥 Commits

Reviewing files that changed from the base of the PR and between 5b35f2e and d68ab89.

📒 Files selected for processing (2)
  • app/Views/utenti/dettagli_utente.php
  • tests/issue-255-registration.spec.js

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@fabiodalez-dev

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

fabiodalez-dev added a commit that referenced this pull request Aug 22, 2026
Release 0.7.64: multiple copies per borrower (#238) + circulation durability, NCIP hardening, verified-release pipeline. Consolidates #368/#370/#371 with F1/F2 review fixes and 25 new tests.
@fabiodalez-dev

Copy link
Copy Markdown
Owner Author

Shipped in v0.7.64: the stored gender enum now renders through the translated labels on the admin user-details page. Consolidated into the release via #372. Closing as merged.

@fabiodalez-dev
fabiodalez-dev deleted the fix-gender-display-admin-user-details branch August 22, 2026 09:56
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