refactor(dashboard): replace period selector with range preset picker

- Swap GoupixDexDashboardPeriodSelect for GoupixDexDashboardRangePresetSelect
- Introduce dashboardRange utility for range preset logic
- Update API stats route and service to support new range handling
- Refactor dashboard charts, useStats composable, and dashboard page
- Sync UI prefs local storage and default layout for new range flows

Co-Authored-By: Oz <oz-agent@warp.dev>

Author: Leoglme

api/routes/stats_route.py

@@ -33,10 +33,6 @@ def _parse_iso(value: str | None) -> dt.datetime | None:
 def dashboard(
     db: Annotated[Session, Depends(get_db)],
     user: Annotated[User, Depends(get_current_user)],
-    include_market: bool = Query(
-        False,
-        description="If true, sums Cardmarket EUR per article via PokéWallet (slower).",
-    ),
     start: str | None = Query(None, description="Range start (ISO date or datetime)."),
     end: str | None = Query(None, description="Range end (ISO date or datetime)."),
     period: Literal["daily", "weekly", "monthly"] = Query(
@@ -47,7 +43,6 @@ def dashboard(
     return compute_dashboard_stats(
         db,
         user.id,
-        include_market=include_market,
         range_start=_parse_iso(start),
         range_end=_parse_iso(end),
         period=period,

api/services/stats_service.py

@@ -9,8 +9,6 @@
 
 from models.article import Article
 from services import article_service
-from services.pricing_service import fetch_card_prices
-
 Period = Literal["daily", "weekly", "monthly"]
 
 
@@ -88,7 +86,6 @@ def compute_dashboard_stats(
     db: Session,
     user_id: int,
     *,
-    include_market: bool = False,
     range_start: dt.datetime | None = None,
     range_end: dt.datetime | None = None,
     period: Period = "daily",
@@ -168,10 +165,10 @@ def in_range(a: Article) -> bool:
         if p is not None
     )
 
-    top_profitable = sorted(sold, key=_profit_eur, reverse=True)[:5]
+    top_profitable = sorted(sold, key=_profit_eur, reverse=True)
     with_duration = [(a, _hours_to_sell(a)) for a in sold]
     with_duration = [(a, h) for a, h in with_duration if h is not None]
-    fastest = sorted(with_duration, key=lambda x: x[1])[:5]
+    fastest = sorted(with_duration, key=lambda x: x[1])
 
     unsold = [a for a in rows if not a.is_sold]
     inventory_count = len(unsold)
@@ -227,25 +224,6 @@ def in_range(a: Article) -> bool:
             }
         )
 
-    market_sum: float | None = None
-    market_sum_unsold: float | None = None
-    market_errors = 0
-    if include_market:
-        market_sum = 0.0
-        market_sum_unsold = 0.0
-        for a in rows:
-            if not a.set_code or not a.card_number:
-                continue
-            p = fetch_card_prices(a.set_code, a.card_number, a.pokemon_name)
-            if p.get("error"):
-                market_errors += 1
-            cm = p.get("cardmarket_eur")
-            if cm is not None:
-                v = float(cm)
-                market_sum += v
-                if not a.is_sold:
-                    market_sum_unsold += v
-
     return {
         "range": {
             "start": range_start.isoformat(),
@@ -273,11 +251,9 @@ def in_range(a: Article) -> bool:
         "inventory_purchase_total_eur": round(inventory_purchase_total_eur, 2),
         "inventory_sell_total_eur": round(inventory_sell_total_eur, 2),
         "inventory_estimated_profit_eur": round(inventory_estimated_profit_eur, 2),
-        "estimated_cardmarket_inventory_eur": round(market_sum, 2) if market_sum is not None else None,
-        "estimated_cardmarket_unsold_eur": round(market_sum_unsold, 2)
-        if market_sum_unsold is not None
-        else None,
-        "market_lookup_errors": market_errors,
+        "estimated_cardmarket_inventory_eur": None,
+        "estimated_cardmarket_unsold_eur": None,
+        "market_lookup_errors": 0,
         "revenue_timeline": revenue_timeline,
         "recent_sales": recent_sales_payload,
         "top_profitable": [

web/app/components/dashboard/GoupixDexCharts.client.vue

@@ -3,9 +3,12 @@
     <UCard>
       <template #header>
         <p class="text-highlighted text-sm font-medium">Cartes les plus rentables</p>
-        <p class="text-muted text-xs">Marge par article (toutes périodes)</p>
+        <p class="text-muted text-xs">Toutes vos ventes classées par marge décroissante</p>
       </template>
-      <ul v-if="stats?.top_profitable?.length" class="divide-default divide-y">
+      <ul
+        v-if="stats?.top_profitable?.length"
+        class="divide-default max-h-80 divide-y overflow-y-auto overscroll-contain"
+      >
         <li v-for="r in stats.top_profitable" :key="r.article_id" class="flex justify-between gap-2 py-2.5 text-sm">
           <span class="text-highlighted truncate">{{ r.title }}</span>
           <span class="text-primary shrink-0 font-semibold">{{ eur.format(r.profit_eur) }}</span>
@@ -17,9 +20,12 @@
     <UCard>
       <template #header>
         <p class="text-highlighted text-sm font-medium">Ventes les plus rapides</p>
-        <p class="text-muted text-xs">Délai entre mise en ligne et vente</p>
+        <p class="text-muted text-xs">Toutes vos ventes avec délai le plus court en premier</p>
       </template>
-      <ul v-if="stats?.fastest_sold?.length" class="divide-default divide-y">
+      <ul
+        v-if="stats?.fastest_sold?.length"
+        class="divide-default max-h-80 divide-y overflow-y-auto overscroll-contain"
+      >
         <li v-for="r in stats.fastest_sold" :key="r.article_id" class="flex justify-between gap-2 py-2.5 text-sm">
           <span class="truncate">{{ r.title }}</span>
           <span class="text-muted shrink-0 tabular-nums">{{ formatTimeToSell(r.hours_to_sell) }}</span>

web/app/components/dashboard/GoupixDexDashboardPeriodSelect.vue

@@ -0,0 +1,55 @@
+<template>
+  <USelect
+    :model-value="selected"
+    :items="items"
+    value-key="value"
+    variant="ghost"
+    class="data-[state=open]:bg-elevated min-w-[12rem]"
+    :ui="{ trailingIcon: 'group-data-[state=open]:rotate-180 transition-transform duration-200' }"
+    @update:model-value="onSelect"
+  />
+</template>
+
+<script setup lang="ts">
+import type { ComputedRef } from 'vue'
+import type { DashboardRange } from '~/composables/useStats'
+import {
+  dashboardRangeForPreset,
+  detectDashboardRangePreset,
+  type DashboardRangePresetId,
+} from '~/utils/dashboardRange'
+
+const range = defineModel<DashboardRange>({ required: true })
+
+type Item = { value: DashboardRangePresetId; label: string; disabled?: boolean }
+
+const BASE_ITEMS: Item[] = [
+  { value: 'week', label: 'Cette semaine' },
+  { value: 'month', label: 'Ce mois-ci' },
+  { value: 'days30', label: '30 derniers jours' },
+  { value: 'months3', label: '3 derniers mois' },
+  { value: 'all', label: 'Depuis toujours' },
+]
+
+const selected: ComputedRef<DashboardRangePresetId> = computed(() => detectDashboardRangePreset(range.value))
+
+const items: ComputedRef<Item[]> = computed(() => {
+  const base = [...BASE_ITEMS]
+  if (selected.value === 'custom') {
+    base.push({ value: 'custom', label: 'Personnalisé', disabled: true })
+  }
+  return base
+})
+
+function onSelect(value: unknown): void {
+  if (value === 'custom' || typeof value !== 'string') {
+    return
+  }
+  const id = value as Exclude<DashboardRangePresetId, 'custom'>
+  const allowed: Exclude<DashboardRangePresetId, 'custom'>[] = ['week', 'month', 'days30', 'months3', 'all']
+  if (!allowed.includes(id)) {
+    return
+  }
+  range.value = dashboardRangeForPreset(id)
+}
+</script>

web/app/components/dashboard/GoupixDexDashboardRangePresetSelect.vue

@@ -78,7 +78,6 @@ export interface DashboardStats {
 }
 
 export interface FetchDashboardOptions {
-  includeMarket?: boolean
   range?: DashboardRange
   period?: DashboardPeriod
 }
@@ -97,21 +96,19 @@ function toIsoDate(date: Date): string {
 /**
  * Dashboard aggregates (`GET /stats/dashboard`).
  *
- * @returns `fetchDashboard` with optional range / period / Cardmarket enrichment flag.
+ * @returns `fetchDashboard` with optional range and timeline bucket size.
  */
 export function useStats() {
   const { $api } = useNuxtApp()
 
   /**
    * Load dashboard KPIs for the selected window.
    *
-   * @param options - Optional custom range, aggregation period, and market-valuation toggle.
+   * @param options - Optional custom range and aggregation period for the revenue timeline.
    * @returns {Promise<DashboardStats>} Full dashboard payload from the API.
    */
   async function fetchDashboard(options: FetchDashboardOptions = {}) {
-    const params: Record<string, string | boolean> = {
-      include_market: options.includeMarket ?? false,
-    }
+    const params: Record<string, string> = {}
     if (options.range) {
       params.start = toIsoDate(options.range.start)
       params.end = toIsoDate(options.range.end)

web/app/composables/useStats.ts

@@ -2,7 +2,7 @@
  * Persistent UI preferences (localStorage), client-only.
  */
 
-import type { DashboardPeriod, DashboardRange } from '~/composables/useStats'
+import type { DashboardRange } from '~/composables/useStats'
 
 const ARTICLES_LIST_KEY = 'goupix_articles_list_prefs'
 const DASHBOARD_KEY = 'goupix_dashboard_prefs'
@@ -76,8 +76,6 @@ export function saveArticleListPrefs(prefs: Partial<ArticleListPrefs>): void {
 
 export interface DashboardPrefsPersisted {
   range: { startIso: string; endIso: string }
-  period: DashboardPeriod
-  fetchMarketData: boolean
 }
 
 /**
@@ -108,12 +106,6 @@ export function loadDashboardPrefs(): Partial<DashboardPrefsPersisted> | null {
         endIso: (p.range as { endIso: string }).endIso,
       }
     }
-    if (p.period === 'daily' || p.period === 'weekly' || p.period === 'monthly') {
-      out.period = p.period
-    }
-    if (typeof p.fetchMarketData === 'boolean') {
-      out.fetchMarketData = p.fetchMarketData
-    }
     return Object.keys(out).length ? out : null
   } catch {
     return null
@@ -123,14 +115,10 @@ export function loadDashboardPrefs(): Partial<DashboardPrefsPersisted> | null {
 /**
  * Persist dashboard UI preferences (range as ISO strings).
  *
- * @param prefs - Selected date range, stats period, and whether to include Cardmarket lookups.
+ * @param prefs - Selected date range for dashboard KPIs.
  * @returns {void} Nothing.
  */
-export function saveDashboardPrefs(prefs: {
-  range: DashboardRange
-  period: DashboardPeriod
-  fetchMarketData: boolean
-}): void {
+export function saveDashboardPrefs(prefs: { range: DashboardRange }): void {
   if (!import.meta.client) {
     return
   }
@@ -140,8 +128,6 @@ export function saveDashboardPrefs(prefs: {
         startIso: prefs.range.start.toISOString(),
         endIso: prefs.range.end.toISOString(),
       },
-      period: prefs.period,
-      fetchMarketData: prefs.fetchMarketData,
     }
     localStorage.setItem(DASHBOARD_KEY, JSON.stringify(payload))
   } catch {

web/app/composables/useUiPrefsLocalStorage.ts

@@ -120,14 +120,6 @@ const links: ComputedRef<NavigationMenuItem[][]> = computed(() => {
         open.value = false
       },
     },
-    {
-      label: 'Journal des publications',
-      icon: 'i-lucide-scroll-text',
-      to: '/articles/listing-logs',
-      onSelect: () => {
-        open.value = false
-      },
-    },
     {
       label: 'Prix du marché',
       icon: 'i-lucide-trending-up',