packages/features/insights/server/__tests__/events.test.ts (1)

803-858: Update tests to use direct formatPeriod import

The tests at the bottom of events.test.ts import formatPeriod directly but still call EventsInsights.formatPeriod. Replace these calls for consistency and to avoid runtime errors.

Affected locations in packages/features/insights/server/tests/events.test.ts:

• Month View “without year” test
• Month View “with year” test
• Year View “same year” test
• Year View “different years” test
• Invalid View test

-        const result = EventsInsights.formatPeriod({
+        const result = formatPeriod({

apps/web/app/(use-page-wrapper)/settings/(admin-layout)/admin/playground/bookings-by-hour/page.tsx (1)

40-44: Hard-coded UI copy should be wrapped with t()

Strings like "Bookings by Hour Playground" and the paragraph on line 42 bypass the i18n layer. Per project guidelines every visible string in .tsx must use t().

-<h1 className="text-3xl font-bold">Bookings by Hour Playground</h1>
+<h1 className="text-3xl font-bold">{t("bookings_by_hour_playground_title")}</h1>

-<p className="mt-2 text-gray-600">
-  This page demonstrates the BookingsByHourChartContent component with sample data.
-</p>
+<p className="mt-2 text-gray-600">
+  {t("bookings_by_hour_playground_description")}
+</p>

apps/web/app/(use-page-wrapper)/settings/(admin-layout)/admin/playground/routing-funnel/page.tsx (1)

61-64: Localize visible text

Lines 61-64 embed plain English copy. Please wrap with t() keys for consistency with localization policy.

-<h1 className="text-3xl font-bold">Routing Funnel Playground</h1>
+<h1 className="text-3xl font-bold">{t("routing_funnel_playground_title")}</h1>

-<p className="mt-2 text-gray-600">
-  This page demonstrates the RoutingFunnelContent component with sample data.
-</p>
+<p className="mt-2 text-gray-600">
+  {t("routing_funnel_playground_description")}
+</p>

packages/features/insights/components/booking/MostCancelledBookingsTables.tsx (1)

9-9: Consider renaming component to match its singular functionality.

The component name MostCancelledBookingsTables uses plural "Tables" but only renders a single UserStatsTable. Consider renaming to MostCancelledBookingsTable for consistency.

-export const MostCancelledBookingsTables = () => {
+export const MostCancelledBookingsTable = () => {

packages/features/insights/hooks/useInsightsBookingParameters.ts (2)

23-36: Consider performance optimization for date calculations.

The date calculations using dayjs are wrapped in useMemo which is good, but in performance-critical scenarios with frequent re-renders, consider using native Date operations or dayjs .utc() methods as they are more performant.

---

20-22: Track the timezone handling refactor.

The TODO indicates an important architectural improvement to move timezone handling to the DateRangeFilter component. This would simplify both frontend and backend code.

Would you like me to create a GitHub issue to track this technical debt item about moving timezone handling to DateRangeFilter?

packages/features/insights/HOW_TO_ADD_BOOKING_CHARTS.md (1)

9-11: Add language specifier to fenced code block.

The fenced code block is missing a language specifier. While it's a text diagram, adding text as the language specifier improves markdown rendering consistency.

-```
+```text
 UI Component → tRPC Handler → Insights Service → Database Query → Response

</blockquote></details>
<details>
<summary>packages/lib/server/service/insightsBooking.ts (1)</summary><blockquote>

`23-24`: **Address TODO: Migrate to AVATAR_FALLBACK constant.**

The TODO indicates a planned migration from the avatar.png endpoint.


Would you like me to create an issue to track migrating from the `avatar.png` endpoint to using the `AVATAR_FALLBACK` constant?

</blockquote></details>

</blockquote></details>

<!-- This is an auto-generated comment by CodeRabbit for review status -->