Skip to content

fix(products): translate category status labels - #1506

Open
khaira777 wants to merge 2 commits into
mainfrom
fix/category-status-i18n
Open

khaira777 wants to merge 2 commits into
mainfrom
fix/category-status-i18n

Conversation

@khaira777

Copy link
Copy Markdown
Contributor

Related work

Issue / Discussion: None (trivial fix; audit finding F-18)

Summary

The Categories table on the Products page rendered hardcoded English 'Active' / 'Inactive'. It now uses tCommon('active') / tCommon('inactive'), as the products table in the same file already does. Both keys exist in every locale file.

Scope

Only the Categories status cell.

Verification

Commands run:

  • npm run lint, npm run build:frontend, npm run i18n:check
  • oxfmt --check, git diff --check

Results: all passed.

Compatibility & data safety

  • Database/data impact: None
  • Migration: None
  • User-visible behavior: The Categories status column is translated in non-English UI languages
  • Offline/network impact: None

Cross-cutting surfaces

Surface Status
UI affected
RTL not affected (text only)
API not affected
Persistence not affected

Developer documentation

  • No developer documentation change needed
  • Existing developer documentation updated
  • New page justified
  • Not applicable (docs-only or code-only change)

Contributor checklist

  • This is a small isolated fix/doc update OR follows an approved issue/direction.
  • Unrelated cleanups, refactors, and dependency changes were kept out of this PR.
  • Tests were added or updated for any behavior changes. (No automated test; see Verification.)
  • Verification commands were run and documented above.
  • Customer data and upgrade safety were preserved (if touching database/storage). (N/A)
  • Documentation or translations were updated where relevant.
  • I understand and can explain all submitted changes.

The Categories table rendered hardcoded English Active/Inactive. Use the existing common.active and common.inactive keys, as the products table already does.
@coderabbitai

coderabbitai Bot commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Translate category status labels on the Products page

🐞 Bug fix 🕐 Less than 5 minutes

Grey Divider

AI Description

• Replace hardcoded English category status labels with existing common translations.
• Show localized Active and Inactive labels without changing category behavior.
High-Level Assessment

Reusing the existing common status keys is the simplest approach and keeps category labels consistent with product labels. New category-specific keys would duplicate translations without adding value.

Files changed (1) +1 / -1

Bug fix (1) +1 / -1
page.tsxLocalize category status text +1/-1

Localize category status text

• The Categories table now displays its active and inactive status labels through the existing common translations instead of hardcoded English text.

frontend/src/app/(dashboard)/products/page.tsx

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can hide the parts of a finding you never read, like the evidence or the agent prompt

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@khaira777
khaira777 enabled auto-merge (squash) November 7, 2026 18:23
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.

1 participant