Title localized#217
Conversation
Author groups — CMS UI, invites, notifications, and permissions
- integrated Tolgee for improved language handling in dashboard components - updated displayDashboardItemTitle to accept localeLanguage for localized titles - modified mapDashboardItemToTableRow to utilize localeLanguage - added tests to verify title selection based on current UI locale - streamlined metadata title selection logic for better fallback handling
Confidence Score: 3/5The locale-to-language mapping and fallback logic are correct and well-tested, but switching UI language will issue a redundant network request and momentarily blank the dashboard table every time. The core title-localization logic in dashboardTable.ts and dashboardApi.ts is clean and covered by tests. The problem is in how localeLanguage is wired into the query in Dashboard.tsx: it enters the query key but is never sent to the server, so any language toggle causes an unnecessary full round-trip and a visible loading state on a page that already has correct data in cache. Dashboard.tsx — specifically how localeLanguage participates in the React Query key and fetch cycle. Reviews (1): Last reviewed commit: "feat(dashboard): enhance localization su..." | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| }; | ||
|
|
||
| return { |
There was a problem hiding this comment.
| }; | |
| return { | |
| }; | |
| return { |
2 participants
No description provided.