Skip to content

fix: make shell overview polling single-flight - #50

Closed
albertovincenzi wants to merge 1 commit into
fix/single-flight-console-pollingfrom
fix/single-flight-overview-polling
Closed

fix: make shell overview polling single-flight#50
albertovincenzi wants to merge 1 commit into
fix/single-flight-console-pollingfrom
fix/single-flight-overview-polling

Conversation

@albertovincenzi

Copy link
Copy Markdown
Collaborator

Dipendenza

Questa PR è impilata su #19 (fix/single-flight-console-polling), che introduce il composable single-flight e la cancellazione su unmount. Va valutata dopo quella PR; il diff specifico qui riguarda soltanto App.vue.

Problema

La shell della console interrogava /api/overview con un setInterval proprio, fuori da usePoll:

  • il timer non veniva mai cancellato;
  • una risposta più lenta di 15 secondi poteva sovrapporsi alla richiesta successiva;
  • il ritorno alla tab visibile non beneficiava del refresh immediato già standardizzato nelle altre viste.

Anche dopo #19, questo era l'unico polling della console che saltava il gate single-flight.

Fix

  • usa usePoll(load, 15000) anche per l'overview della shell;
  • dopo fetchMe, invoca il refresh restituito dal composable;
  • elimina il setInterval manuale.

Il primo tick montato prima dell'autenticazione è innocuo (load esce su authState !== ready); il refresh dopo fetchMe garantisce il caricamento immediato. Se i due eventi coincidono, il single-flight di #19 accoda al massimo un solo follow-up.

Verifica

  • cd ui && npm run build (52 moduli trasformati, build Vite riuscita);
  • git diff --check.

Per Alice

Dopo il merge di #19, il criterio manuale è semplice: aprire la console, verificare l'overview del broker dopo il login, nascondere/mostrare la tab e controllare un refresh immediato senza richieste concorrenti nel pannello Network.

@alice-viola

Copy link
Copy Markdown
Contributor

Landed on master via #67 (merge commit 944ee9b) as part of the 62-PR integration — this PR's head commit 8b4c098 is an ancestor of master. GitHub could not mark it merged automatically because its base is fix/single-flight-console-polling, not master. Closing as landed.

@alice-viola alice-viola closed this Sep 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants