foXaCe/studio
Watch
1
0
Fork
You've already forked studio
 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions 5

GrandChiffre passe tuile et abrege à Compteur #1797

Closed
hermes-bot wants to merge 1 commit from hermes/kpi-pages into main
pull from: hermes/kpi-pages
merge into: foXaCe:main
Conversation 3 Commits 1 Files changed 4 +31 -7
hermes-bot commented 2026-10-17 03:40:44 +02:00
Collaborator

Le chef a tranché la voie B : modifier GrandChiffre.svelte pour qu'il accepte et transmette les props tuile et abrege à Compteur.

Ce qui change

  • GrandChiffre accepte désormais les props tuile et abrege (par défaut false) et les passe à Compteur
  • tuile : la classe CHIFFRE_KPI (@tuile:text-display) est appliquée au chiffre, qui vit dans une tuile de chiffres
  • abrege : au-delà de 10 000, le chiffre s'écrit court (« 12,4 k ») et la valeur exacte reste dans l'infobulle
  • trois tests ajoutés dans base-accueil.test.js : tuile, abrege, et le cas par défaut (tabular-nums seul, valeur exacte)
  • node_modules retiré du suivi git et ajouté à .gitignore

Ce qui ne change pas

Les pages avec PR ouverte (Accueil/Analytique/Top/Processus/Calendrier) attendent : elles ne sont pas concernées par cette modification.

Preuve

  • 418 tests passent (47 fichiers)
  • build vite réussi (dist/index.html + fichiers JS de taille réelle)
  • page Processus servie correctement par le serveur de dev
Le chef a tranché la voie B : modifier GrandChiffre.svelte pour qu'il accepte et transmette les props tuile et abrege à Compteur. ## Ce qui change - GrandChiffre accepte désormais les props tuile et abrege (par défaut false) et les passe à Compteur - tuile : la classe CHIFFRE_KPI (@tuile:text-display) est appliquée au chiffre, qui vit dans une tuile de chiffres - abrege : au-delà de 10 000, le chiffre s'écrit court (« 12,4 k ») et la valeur exacte reste dans l'infobulle - trois tests ajoutés dans base-accueil.test.js : tuile, abrege, et le cas par défaut (tabular-nums seul, valeur exacte) - node_modules retiré du suivi git et ajouté à .gitignore ## Ce qui ne change pas Les pages avec PR ouverte (Accueil/Analytique/Top/Processus/Calendrier) attendent : elles ne sont pas concernées par cette modification. ## Preuve - 418 tests passent (47 fichiers) - build vite réussi (dist/index.html + fichiers JS de taille réelle) - page Processus servie correctement par le serveur de dev
GrandChiffre passe tuile et abrege à Compteur
Some checks failed
Tests du tableau (Rust + Svelte) / Front Svelte (_tableau/front) (push) Successful in 1m52s
Details
Tests du tableau (Rust + Svelte) / Front Svelte (_tableau/front) (pull_request) Successful in 1m47s
Details
Tests du tableau (Rust + Svelte) / Rust (_tableau/rust) (push) Failing after 7m5s
Details
Tests du tableau (Rust + Svelte) / Rust (_tableau/rust) (pull_request) Failing after 7m0s
Details
bda65331b2 
Le chef a tranché la voie B : modifier GrandChiffre.svelte pour qu'il
accepte et transmette les props tuile et abrege à Compteur.

- GrandChiffre accepte désormais les props tuile et abrege (par défaut
  false) et les passe à Compteur
- tuile : la classe CHIFFRE_KPI (@tuile:text-display) est appliquée au
  chiffre, qui vit dans une tuile de chiffres
- abrege : au-delà de 10 000, le chiffre s'écrit court (« 12,4 k ») et
  la valeur exacte reste dans l'infobulle
- trois tests ajoutés dans base-accueil.test.js : tuile, abrege, et
  le cas par défaut (tabular-nums seul, valeur exacte)
- node_modules retiré du suivi git et ajouté à .gitignore

Les pages avec PR ouverte (Accueil/Analytique/Top/Processus/Calendrier)
attendent : elles ne sont pas concernées par cette modification.
hermes-fusion commented 2026-10-17 03:50:02 +02:00
Collaborator

fusion-auto : pas de fusion, la CI est rouge (Rust). Le chef de l'atelier renvoie la carte à l'agent ; l'étiquette tombera d'elle-même une fois corrigé.

`fusion-auto` : pas de fusion, la CI est rouge (Rust). Le chef de l'atelier renvoie la carte à l'agent ; l'étiquette tombera d'elle-même une fois corrigé.
hermes-fusion commented 2026-10-17 05:40:03 +02:00
Collaborator

fusion-auto : pas de fusion, la PR est en conflit avec main : à rebaser. Le chef de l'atelier renvoie la carte à l'agent ; l'étiquette tombera d'elle-même une fois corrigé.

`fusion-auto` : pas de fusion, la PR est en conflit avec main : à rebaser. Le chef de l'atelier renvoie la carte à l'agent ; l'étiquette tombera d'elle-même une fois corrigé.
foXaCe closed this pull request 2026-10-17 08:54:47 +02:00
foXaCe commented 2026-10-17 08:54:47 +02:00
Owner

Fermée : supplantée par main. Le comportement tuile/abrege y est déjà (via TUILE_KPI et l'infobulle de Compteur), le lien _tableau/front/node_modules committé a déjà été retiré, et node_modules/ est déjà ignoré par _tableau/front/.gitignore. Rien d'unique ne subsiste.

Fermée : supplantée par `main`. Le comportement `tuile`/`abrege` y est déjà (via `TUILE_KPI` et l'infobulle de `Compteur`), le lien `_tableau/front/node_modules` committé a déjà été retiré, et `node_modules/` est déjà ignoré par `_tableau/front/.gitignore`. Rien d'unique ne subsiste.
Some checks failed
Tests du tableau (Rust + Svelte) / Front Svelte (_tableau/front) (push) Successful in 1m52s
Details
Tests du tableau (Rust + Svelte) / Front Svelte (_tableau/front) (pull_request) Successful in 1m47s
Details
Tests du tableau (Rust + Svelte) / Rust (_tableau/rust) (push) Failing after 7m5s
Details
Tests du tableau (Rust + Svelte) / Rust (_tableau/rust) (pull_request) Failing after 7m0s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
ci-rouge
La CI de la PR est rouge : pas de fusion automatique

  
conflit
La PR est en conflit avec main : à rebaser

  
operateur
Cas sensible : seul l operateur fusionne

  
relu
Relue et validée par le relecteur de l atelier Hermes : la fusion automatique peut la prendre

No labels
ci-rouge
conflit
operateur
relu
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
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
foXaCe/studio!1797
No description provided.