-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Open
Labels
cleanupRemove non-functional or unnecessary codeRemove non-functional or unnecessary coderefactorRefactoring tasksRefactoring tasksunapproved
Description
Context
As part of the repository-wide translation compliance cleanup (post #7364), this batch addresses Tag and Organization-related components that are currently passing t as a prop/parameter instead of calling useTranslation() locally.
This batch is domain-scoped to keep refactors reviewable and reduce merge conflicts.
Affected Files
Tag-related:
- src/components/AdminPortal/TagActions/TagActions.tsx
- src/components/AdminPortal/TagActions/Node/TagNode.tsx
- src/screens/AdminPortal/ManageTag/editModal/EditUserTagModal.tsx
- src/components/AdminPortal/AddPeopleToTag/AddPeopleToTag.tsx
Organization-related:
- src/components/AdminPortal/OrgList/modal/OrganizationModal.tsx
- src/screens/AdminPortal/OrgList/OrgList.tsx
- src/screens/UserPortal/Organizations/Organizations.tsx
Required Refactor
For each affected file:
- Remove
tfrom component props and parameters - Eliminate
tprop drilling chains - Import
useTranslationfromreact-i18next - Call
const { t } = useTranslation()locally - Ensure correct namespace usage
- Update related tests where
twas mocked or injected
No UI or functional behavior should change.
Acceptance Criteria
- No component in this batch receives
tas a prop - All translation calls originate from local
useTranslation() - Tests pass without flakiness
- No coverage regression
- Translation lint checks pass in CI
- Changes are limited strictly to this domain
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
cleanupRemove non-functional or unnecessary codeRemove non-functional or unnecessary coderefactorRefactoring tasksRefactoring tasksunapproved
Type
Projects
Status
Backlog