chore: remove some code smells

src/components/ui/UserSwitchWrapper.tsx

@@ -16,7 +16,7 @@ type Props = {
 
 export function UserSwitchWrapper({
   ButtonContent,
-}: Props): JSX.Element | null {
+}: Readonly<Props>): JSX.Element | null {
   const { i18n } = useTranslation(NS.Account);
   const { isAuthenticated, user, logout } = useAuth();
 

src/modules/account/home/memberAnalytics/MemberGeneralStatisticsCards.tsx

@@ -45,7 +45,7 @@ type Props = {
 };
 export function MemberGeneralStatisticsCards({
   actionsGroupedByTypes,
-}: Props): JSX.Element {
+}: Readonly<Props>): JSX.Element {
   const { t } = useTranslation(NS.Analytics);
 
   return (

src/modules/analytics/context/MyAnalyticsDateRangeContext.tsx

@@ -1,6 +1,7 @@
 import { createContext, useMemo, useState } from 'react';
 
 import { addDays, intervalToDuration } from 'date-fns';
+import type { Duration } from 'date-fns';
 
 import { DateRange, GroupByInterval } from '~analytics/config/type';
 
@@ -22,6 +23,20 @@ const defaultValue: {
 
 export const MyAnalyticsDateRangeDataContext = createContext(defaultValue);
 
+function getGroupInterval(duration: Duration) {
+  if (duration.years && duration.years >= 1) {
+    return GroupByInterval.Year;
+  }
+  if (duration.months && duration.months > 2) {
+    return GroupByInterval.Month;
+  }
+  if (duration.days && duration.days < 8) {
+    return GroupByInterval.Day;
+  } else {
+    return GroupByInterval.Week;
+  }
+}
+
 const MyAnalyticsDateRangeProvider = ({
   children,
 }: {
@@ -33,19 +48,12 @@ const MyAnalyticsDateRangeProvider = ({
     key: 'selection',
   });
 
-  const { months, days, years } = intervalToDuration({
-    start: dateRange.startDate,
-    end: dateRange.endDate,
-  });
-
-  const groupInterval =
-    years && years >= 1
-      ? GroupByInterval.Year
-      : months && months > 2
-        ? GroupByInterval.Month
-        : days && days < 8
-          ? GroupByInterval.Day
-          : GroupByInterval.Week;
+  const groupInterval = getGroupInterval(
+    intervalToDuration({
+      start: dateRange.startDate,
+      end: dateRange.endDate,
+    }),
+  );
 
   const value = useMemo(
     () => ({

src/modules/analytics/layout/AnalyticsSidebar.tsx

@@ -24,7 +24,9 @@ import {
 } from '~analytics/config/selectors';
 import { DataContext } from '~analytics/context/DataProvider';
 
-export function AnalyticsSidebar({ itemId }: { itemId: string }): JSX.Element {
+export function AnalyticsSidebar({
+  itemId,
+}: Readonly<{ itemId: string }>): JSX.Element {
   const { t } = useTranslation(NS.Analytics);
   const { descendantApps } = useContext(DataContext);
   const { data: item } = hooks.useItem(itemId);

src/modules/analytics/layout/Navigator.tsx

@@ -18,7 +18,9 @@ import {
 
 const { useItem, useParents, useCurrentMember, useChildren } = hooks;
 
-export function Navigator({ itemId }: { itemId: string }): JSX.Element | null {
+export function Navigator({
+  itemId,
+}: Readonly<{ itemId: string }>): JSX.Element | null {
   const { data: currentMember } = useCurrentMember();
   const { data: item, isLoading: isItemLoading } = useItem(itemId);