dashboard : écran Site — fiche, courbe, recommandations, traçabilité (#24) #140

Merged

justine merged 4 commits from marvin/24-ecran-site into develop 2026-09-07 09:01:43 +00:00

Conversation 3 Commits 4 Files changed 19 +2124 -8

marvin commented 2026-09-04 08:01:07 +00:00

Member

Copy link

Ce que ça change

L'écran d'un seul site sur /sites/:id : fiche (référentiel et état courant), courbe avec sa référence en pointillé et la prévision H+1, recommandations, et traçabilité d'un point jusqu'à la lecture bronze (EF-04) en cliquant une barre du graphique. Quatre widgets, quatre statuts indépendants — une fiche en échec ne bloque ni la courbe ni les recommandations.

Les points imputés se distinguent en permanence sur la courbe (teinte allégée + légende), pas seulement au survol : c'est le manque laissé sur l'écran Parc, corrigé ici.

Base : marvin/24-ecran-parc (PR #139), dont cet écran dépend. Dernier des trois lots front extraits de #127, à merger en dernier.

Refs #24

Preuve

207 tests vitest passes (43 neufs)
build de production propre

Ecran Site -- fiche du site, courbe 24 h avec sa ligne de reference en pointille
(758 kW), legende mesure / reconstitue (EF-04), prevision de l'heure suivante et
son ecart, recommandations a droite.

Tracabilite au clic -- cliquer une barre remonte la valeur jusqu'a sa lecture
bronze (EF-04) : instant, valeur affichee, valeur lue, methode de reconstitution et
qualite. Ici une mesure directe, sans reconstitution.

Relecture

• Un pair a relu et laissé un commentaire, même court
• Ses remarques sont traitées, ou une réponse explique pourquoi elles ne le sont pas

Ce qui suit le code

(rien de coché : aucun runbook, ADR, incident ni variable d'environnement nouvelle)

Où regarder en priorité

• La traçabilité est branchée sur le point qu'on regarde déjà, pas sur une route à deviner : chaque barre est cliquable et charge /v1/sites/{id}/tracabilite pour son horodatage. L'API rend 404 sur un instant qui ne correspond à aucun point de la fenêtre 24 h — à vérifier que le comportement au clic reste lisible dans ce cas.
• SiteView recharge tout au changement de :id via un watch, pas seulement onMounted : vue-router réutilise l'instance du composant entre deux sites de la même route nommée.
• Les trois routes ajoutées ici visent /v1/… comme les précédentes (voir PR #139 pour le pourquoi du préfixe).

Après cette PR, il reste à #24 : l'écran Qualité (ses routes existent, PR #137), l'affichage des alertes, la garde de route sur 401, axe-core, les tests e2e et la vérification aux trois largeurs.

## Ce que ça change L'écran d'un seul site sur `/sites/:id` : fiche (référentiel et état courant), courbe avec sa référence en pointillé et la prévision H+1, recommandations, et traçabilité d'un point jusqu'à la lecture bronze (EF-04) en cliquant une barre du graphique. Quatre widgets, quatre statuts indépendants — une fiche en échec ne bloque ni la courbe ni les recommandations. Les points imputés se distinguent **en permanence** sur la courbe (teinte allégée + légende), pas seulement au survol : c'est le manque laissé sur l'écran Parc, corrigé ici. **Base : `marvin/24-ecran-parc`** (PR #139), dont cet écran dépend. Dernier des trois lots front extraits de #127, à merger en dernier. Refs #24 ## Preuve ``` 207 tests vitest passes (43 neufs) build de production propre ``` **Ecran Site** -- fiche du site, courbe 24 h avec sa ligne de reference en pointille (758 kW), legende `mesure` / `reconstitue (EF-04)`, prevision de l'heure suivante et son ecart, recommandations a droite.  **Tracabilite au clic** -- cliquer une barre remonte la valeur jusqu'a sa lecture bronze (EF-04) : instant, valeur affichee, valeur lue, methode de reconstitution et qualite. Ici une mesure directe, sans reconstitution.  ## Relecture - [ ] Un pair a relu et laissé un commentaire, même court - [ ] Ses remarques sont traitées, ou une réponse explique pourquoi elles ne le sont pas ## Ce qui suit le code (rien de coché : aucun runbook, ADR, incident ni variable d'environnement nouvelle) ## Où regarder en priorité - **La traçabilité est branchée sur le point qu'on regarde déjà**, pas sur une route à deviner : chaque barre est cliquable et charge `/v1/sites/{id}/tracabilite` pour son horodatage. L'API rend 404 sur un instant qui ne correspond à aucun point de la fenêtre 24 h — à vérifier que le comportement au clic reste lisible dans ce cas. - `SiteView` recharge tout au changement de `:id` via un `watch`, pas seulement `onMounted` : vue-router réutilise l'instance du composant entre deux sites de la même route nommée. - Les trois routes ajoutées ici visent `/v1/…` comme les précédentes (voir PR #139 pour le pourquoi du préfixe). **Après cette PR, il reste à #24** : l'écran Qualité (ses routes existent, PR #137), l'affichage des alertes, la garde de route sur 401, axe-core, les tests e2e et la vérification aux trois largeurs.

140-site-fiche-courbe-recommandations.jpg

61 KiB

140-site-tracabilite-ef04.jpg

73 KiB

marvin added 1 commit 2026-09-04 08:01:08 +00:00

dashboard : écran Site — fiche, courbe, recommandations, traçabilité (ticket #24) ... 8dad3c2b63

Nouvel écran sur /sites/:id : FicheSite (référentiel et état courant),
GraphiqueSite (une série, sa référence en pointillé, prévision H+1 et son
écart), RecommandationsSite, TracabiliteDetail. Quatre widgets, quatre
statuts indépendants (stores/site.js) — une fiche en échec ne bloque pas
la courbe ni les recommandations.

Chaque barre de GraphiqueSite est cliquable : elle charge la traçabilité
de ce point (EF-04, « une valeur jusqu'à la lecture bronze »), affichée
par TracabiliteDetail à côté — pas une route à deviner, le point qu'on
regarde déjà. Les points imputés se distinguent en permanence sur la
courbe (teinte allégée + légende), pas seulement au survol : le manque
laissé sur l'écran Parc est corrigé ici.

Intégration : le nom d'un site dans le tableau du Parc mène désormais à
sa fiche.

Les trois routes ajoutées ici (fiche, recommandations, traçabilité) visent
« /v1/… » comme les précédentes : le préfixe « /api » appartient à Caddy, qui
le retire avant de relayer, et c'est VITE_API_BASE qui le pose.

43 tests neufs (207 au total pour le front), build de production propre.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01P6BwMkMyRBYJJYaHheLwGY

marvin force-pushed marvin/24-ecran-site from 8dad3c2b63 to c49b00821f 2026-09-04 08:02:07 +00:00 Compare

marvin changed title from dashboard : �cran Site � fiche, courbe, recommandations, tra�abilit� (#24) to dashboard : écran Site — fiche, courbe, recommandations, traçabilité (#24) 2026-09-04 08:04:04 +00:00

marvin self-assigned this 2026-09-04 08:35:48 +00:00

marvin requested review from justine 2026-09-04 08:52:57 +00:00

olivier referenced this pull request 2026-09-04 09:07:46 +00:00

[infra] Chaîne d'intégration : le job du tableau de bord échoue un tiers du temps sur un délai réseau non borné #148

marvin referenced this pull request 2026-09-04 14:19:53 +00:00

dashboard : alertes et journal de collecte de l'écran Qualité (#24) #155

marvin referenced this pull request 2026-09-04 14:25:43 +00:00

dashboard : écran Qualité — disponibilité par site et part de points imputés (#24) #156

marvin referenced this pull request 2026-09-04 14:40:04 +00:00

[Front] Tableau de bord — les trois écrans branchés sur l'API #24

justine added 1 commit 2026-09-07 06:32:09 +00:00

Merge branch 'marvin/24-ecran-parc' into marvin/24-ecran-site 6fbf3e5c3a

justine commented 2026-09-07 07:15:29 +00:00

Member

Copy link

Les critères de l'écran "site" sont validés par contre quelques petits soucis sur le RGAA :

• on ne peut pas atteindre la traçabilité juste avec le clavier (onClick: surClicBarre -> .toile-conteneur est tabindex="0")
• les couleurs trop similaires entre la valeur et la valeur imputée -> COULEUR_SERIE_IMPUTEE = COULEUR_SERIE avec juste 40% + la légende utilise les 2 mêmes pastilles

Autres petits bugs :

• le clic hors "24h" va donner un 404 sur toutes les barres sauf la dernière
• petite incohérence sur l'emplacement des traductions entre LIBELLE_METHODE, LIBELLE_QUALITE et glossaire.js
• comme pour la #139 on utilise pas la bonne unité sur le kw : Consommation -> kWh avec des valeur_kw

Les critères de l'écran "site" sont validés par contre quelques petits soucis sur le RGAA : - on ne peut pas atteindre la traçabilité juste avec le clavier (onClick: surClicBarre -> .toile-conteneur est tabindex="0") - les couleurs trop similaires entre la valeur et la valeur imputée -> COULEUR_SERIE_IMPUTEE = COULEUR_SERIE avec juste 40% + la légende utilise les 2 mêmes pastilles Autres petits bugs : - le clic hors "24h" va donner un 404 sur toutes les barres sauf la dernière - petite incohérence sur l'emplacement des traductions entre LIBELLE_METHODE, LIBELLE_QUALITE et glossaire.js - comme pour la #139 on utilise pas la bonne unité sur le kw : Consommation -> kWh avec des valeur_kw

marvin changed target branch from marvin/24-ecran-parc to develop 2026-09-07 07:37:11 +00:00

marvin referenced this pull request from a commit 2026-09-07 08:50:01 +00:00

dashboard: la traçabilité s'atteint au clavier, et se distingue à l'œil (#24)

marvin added 1 commit 2026-09-07 08:50:01 +00:00

dashboard: la traçabilité s'atteint au clavier, et se distingue à l'œil (#24) ... 3bbc9d28b6

Relecture de la #140.

Accès clavier. La traçabilité se prenait au clic seul. Le conteneur portait
déjà « tabindex="0" » — l'intention était là — mais aucune touche n'était
écoutée : au clavier, la traçabilité était hors d'atteinte. Les flèches
parcourent les barres, Home et Fin mènent aux bouts, Entrée ouvre la
traçabilité de la barre désignée. Un anneau la marque sur le canevas, et une
ligne sous le graphique dit ce qu'elle vaut — l'anneau dit laquelle, jamais
combien.

Valeur reconstituée. Elle se distinguait par la même teinte à 40 %, que la
légende répétait à l'identique. Deux nuances d'une même teinte se distinguent
mal sous faible contraste, et l'aplat clair ne tenait pas les 3:1 attendus
d'un élément graphique. La barre porte maintenant des hachures et un contour :
la forme dit la différence, la couleur ne fait que la redoubler. La pastille
de légende reprend les mêmes hachures.

Traçabilité hors 24 h. L'API ne trace qu'un point de la fenêtre 24 h ; au-delà
le pas est journalier et une barre agrège vingt-quatre relevés. Le clic
partait quand même, et rendait 404 sur toutes les barres sauf la dernière —
son début de journée tombant encore dans les vingt-quatre dernières heures.
On ne le propose plus, et on dit pourquoi, au lieu de le faire découvrir par
une erreur. La condition porte sur le pas rendu par l'API, pas sur la valeur
du sélecteur.

Traductions. LIBELLE_METHODE et LIBELLE_QUALITE vivaient dans
TracabiliteDetail.vue, alors que glossaire.js dit porter la traduction des
valeurs d'énumération « et nulle part ailleurs dans le front ». Elles y
rejoignent les autres, avec le repli des voisines.

Unité. Le titre annonçait des kWh sur des valeur_kw. GraphiqueConsommation.vue
porte la même faute mais n'est pas touché ici : elle est déjà corrigée sur la
branche de la #139, et la corriger deux fois est exactement ce qui a produit
les conflits sur App.vue.

224 tests vitest passent, la construction est propre.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DFGp2MvuHVuoJEss2G5nn1

marvin commented 2026-09-07 08:54:59 +00:00

Author

Member

Copy link

corrigé !

corrigé !

👍 1

justine added 1 commit 2026-09-07 08:56:07 +00:00

Merge branch 'develop' into marvin/24-ecran-site 82de219c22

justine approved these changes 2026-09-07 09:01:36 +00:00

justine merged commit ce58c07e36 into develop 2026-09-07 09:01:43 +00:00

justine deleted branch marvin/24-ecran-site 2026-09-07 09:01:44 +00:00

justine referenced this pull request from a commit 2026-09-07 09:01:45 +00:00

Merge pull request 'dashboard : écran Site — fiche, courbe, recommandations, traçabilité (#24)' (#140) from marvin/24-ecran-site into develop

lenaic referenced this pull request 2026-09-07 13:48:55 +00:00

Jalon : develop vers main, la chaîne complète et l'application sur le serveur #170

Sign in to join this conversation.

Reviewers

No reviewers

justine

Labels

Clear labels   

Compat/Breaking

Breaking change that won't be backward compatible

  

EC01

Preuve pour l'épreuve Architecture

  

EC02

Preuve pour l'épreuve Management

  

EC03

Preuve pour l'épreuve CI/CD

  

EC04

Preuve pour l'épreuve Cloud

  

EC05

Preuve pour l'épreuve Data et BI

  

EC06

Preuve pour l'épreuve IA

  

Kind/BDD

BDD

  

Kind/Back

Back

  

Kind/Bug

Something is not working

  

Kind/CICD

CICD

  

Kind/Cloud

Cloud

  

Kind/Contenu

Contenu

  

Kind/Data

Data

  

Kind/Documentation

Documentation changes

  

Kind/Enhancement

Improve existing functionality

  

Kind/Feature

New functionality

  

Kind/Front

Front

  

Kind/IA

IA

  

Kind/Infra

Serveur, déploiement, chaîne, supervision

  

Kind/Monitoring

Monitoring

  

Kind/Security

This is security issue

  

Kind/Testing

Issue or pull request related to testing

  

Portée/Post-jury

Périmètre documenté, hors de la fenêtre du 11/09. Repris après le jury. Rien nest supprimé.

  

Priority

Critical

The priority is critical

  

Priority

High

The priority is high

  

Priority

Low

The priority is low

  

Priority

Medium

The priority is medium

  

Reviewed

Confirmed

Issue has been confirmed

  

Reviewed

Duplicate

This issue or pull request already exists

  

Reviewed

Invalid

Invalid issue

  

Reviewed

Won't Fix

This issue won't be fixed

  

Status

Abandoned

Somebody has started to work on this but abandoned work

  

Status

Blocked

Something is blocking this issue or pull request

  

Status

Need More Info

Feedback is required to reproduce issue or to continue work

  

ops/alerte

Alerte d'exploitation (relais Grafana, #42)

No labels

Compat/Breaking

EC01

EC02

EC03

EC04

EC05

EC06

Kind/BDD

Kind/Back

Kind/Bug

Kind/CICD

Kind/Cloud

Kind/Contenu

Kind/Data

Kind/Documentation

Kind/Enhancement

Kind/Feature

Kind/Front

Kind/IA

Kind/Infra

Kind/Monitoring

Kind/Security

Kind/Testing

Portée/Post-jury

Priority

Critical

Priority

High

Priority

Low

Priority

Medium

Reviewed

Confirmed

Reviewed

Duplicate

Reviewed

Invalid

Reviewed

Won't Fix

Status

Abandoned

Status

Blocked

Status

Need More Info

ops/alerte

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

marvin

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

g2/enervision!140

No description provided.