controlTempStore.setDeviceStatus mutates state in place, breaking zustand change detection
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Tranquilla
- Stack tecnologico
- javascript, react, typescript
- Ambito
- developer-experience, frontend
Direzione di ricerca
Inizia dalle righe 14–18 di ui/src/pages/ControlTempPage/controlTempStore.tsx e leggi l’aggiornamento dello store Zustand, poi controlla i selettori in AlarmDismissal.tsx e PowerButton.tsx. Sostituisci la fusione in loco con un aggiornamento immutabile che gestisca anche i campi rimossi o svuotati. Il lavoro è completato quando gli iscritti ricevono aggiornamenti al variare dello stato del dispositivo e i valori annidati obsoleti non vengono mantenuti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
ui/src/pages/ControlTempPage/controlTempStore.tsx:14-18 does _.merge(deviceStatus, newDeviceStatus) which mutates the existing store object and returns the same reference, then calls set({ deviceStatus: updatedDeviceStatus }). Components that subscribe with an equality selector (e.g. AlarmDismissal.tsx:33 and PowerButton.tsx:24 via state => state.deviceStatus) will not be re-notified because the referenced object is unchanged (Object.is passes), so they can render stale device status unless a parent re-render happens to propagate. Additionally _.merge deep-merges and never removes keys, so cleared/partial fields can retain stale nested values. Use an immutable update (spread a new object) instead of mutating.
Filed from a repo-wide code audit (working tree at bcad746).
- Lingua principale
- Rust
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 GoogleBot42/podd
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 84/100
GoogleBot42/podd#63 ·
-
documentation
Difficoltà 1/5 1-3 ore Idoneità per principianti 78/100
GoogleBot42/podd#62 ·
-
documentation good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 69/100
GoogleBot42/podd#60 ·
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
GoogleBot42/podd#61 ·
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
GoogleBot42/podd#58 ·
Tutte le issue di GoogleBot42/podd
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
rubys/roundhouse#1917 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 2 giorni
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
[cosmic-applet-battery] Division by zero displays NaN% when UPower reports max_brightness = 0Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
pop-os/cosmic-applets#1619 ·
I maintainer di solito rispondono entro 3 giorni
-
binding:python bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno