Migrate subscription cleanup to takeUntilDestroyed
I maintainer di solito rispondono entro 2 giorni
@bracyw ci sta già lavorando.
Dal 29/3/2026.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
What
Components clean up subscriptions by hand with Subscription[] arrays and ngOnDestroy, and some never clean up at all. Move all of them to takeUntilDestroyed so subscriptions end when the component is destroyed. This also unblocks #798: refcounted storage (#799) can only drop a topic once its subscribers tear down.
Requirements
- Every manual Subscription[] + ngOnDestroy cleanup uses takeUntilDestroyed instead (29 files on develop)
- Components that subscribe with no cleanup now use takeUntilDestroyed
- OnDestroy removed wherever it only existed for subscription cleanup
- No behavior change; lint, tests and build pass
Notes
Nobody has started this; no branch or PR exists. PR #550 already moved the graph sidebars to toSignal, which is fine where it fits.
Generated by Claude Code
- Lingua principale
- TypeScript
- Stelle
- 5
- Fork
- 1
- Merge medio
- 2g 14h
- PR unite (30g)
- 51
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Northeastern-Electric-Racing/Argos
-
Remove the landing pageForse già presa @Idontknow0487 l’ha presa 5 giorni fa. Apertaby: ai-assisted frontend good-for: new-member spike
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Northeastern-Electric-Racing/Argos#847 · 1 assegnatario ·
I maintainer di solito rispondono entro 2 giorni
-
Move the small charts to Chart.js and drop ApexChartsForse già presa @bracyw l’ha presa 1 giorno fa. Apertaby: ai dev work frontend
Northeastern-Electric-Racing/Argos#926 · 1 assegnatario ·
I maintainer di solito rispondono entro 2 giorni
-
Move the graph page to Chart.jsForse già presa @bracyw l’ha presa 1 giorno fa. Apertaby: ai dev work frontend
Northeastern-Electric-Racing/Argos#925 · 1 assegnatario ·
I maintainer di solito rispondono entro 2 giorni
-
Move the client's charts to Chart.jsForse già presa @bracyw l’ha presa 1 giorno fa. Apertaby: ai epic frontend
Northeastern-Electric-Racing/Argos#924 · 1 assegnatario ·
I maintainer di solito rispondono entro 2 giorni
-
quick-view-ownership: Build on the app-wide quick view from #898Forse già presa @bracyw l’ha presa 10 giorni fa. Apertaby: ai frontend spike
Northeastern-Electric-Racing/Argos#904 · 1 assegnatario ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di Northeastern-Electric-Racing/Argos
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
gnosis/gnosis_vpn#661 ·
I maintainer di solito rispondono entro 1 giorno
-
content-freshness: staleOwnedContent doc comments still say 'complete receipt' (follow-up to #7912)Apertatriage:bug-verified
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
I maintainer di solito rispondono entro 1 giorno
-
fix: reject sourceStorage/packagedStorage bucket that differs from the stack default (fail loud)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Eyevinn/open-videocore#1300 ·
I maintainer di solito rispondono entro 1 giorno
-
nextjs-tracking
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
cloudflare/vinext#4390 ·
I maintainer di solito rispondono entro 1 giorno
-
kind/bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
kubernetes-sigs/headlamp#8175 ·
I maintainer di solito rispondono entro 2 giorni