AlfioEmanueleFresta
commented
8 years ago @drinky78 e @pizar, ho inziato a leggere il codice, ci sono un paio di problemi seri con questa pull request che ho bisogno che mi risolviate:
- [ ] Importante: I commit contengono le vostre copie dei file di configurazione, incluse le vostre credenziali database e altro. Questi file sono presenti in
.gitignore, in modo da non essere aggiunti automaticamente, ma e' stato forzato l'inserimento per il commit. Dovreste rimuovere questi file dalla pull request. Esempi: - [ ] Molto importante: Ho paura che i "merge" non siano stati effettuati correttamente. Infatti, mi risulta che invece di aver corretto i conflitti, questi siano stati ignorati e la vostra versione del codice sia stata forzata. Questo vuol dire che ove vi erano dei conflitti in parte dei file, questi sono regrediti ad una versione inconsistente (rotta?) che contiene le vostre modifiche ma che non tiene conto delle modifiche ai file non in comune. Esempio:
- https://github.com/CroceRossaCatania/gaia/pull/2111/files#diff-c0e4599d98bf760d8f0d1cd5e29e3eccL196: in questo file, ad esempio, e' stata forzata la vostra versione, rimuovendo un numero x di modifiche fatte da noi negli ultimi mesi e anche arbitrariamente delle funzioni inserite (es. la funzione relativa al modulo donazione sangue, in fondo).
- [ ] Poco importante
Molto importante: Vi porto ad esempio il commit https://github.com/pizar/gaia/commit/66fe027e1493cd8933746ed93ae5faeaf12d9272, dove ad occhio, sono stati rimossi centinaia di file usati da Gaia, come i modelli per le e-mail. Perche'?- Aggiornamento: Questo mi risulta essere stato annullato in qualche altro commit, ma continua a non essermene chiara la ragione.
/cc @CroceRossaCatania/sviluppatori
Questa pull request introduce le funzionalita' sviluppate da @pizar e @drinky78 a riguardo dell'Accademia di Formazione - Area 1.
Processo di merge: