Code review comment for lp:~pietroalbini/ubuntu-it-ask/fix-lp1197300

Revision history for this message
Milo Casagrande (milo) wrote :

Questa modifica è stata provata in locale o su un'istanza di test?

Quando si tratta di modificare una classe in models/ è sempre meglio controllare due volte. Qui stai modificando la lunghezza del campo nel database. È effettivamente quello che vuoi fare o la soluzione al problema originale?

Potrebbe non essere un problema fare il deploy con questa modifica, sopratutto se l'applicazione usa South per gestire le migrazioni del DB, ma va assolutamente verificato.

review: Needs Information

« Back to merge proposal