Buenas, Dani, Varias cosas:
- Mejor hacer un MP distinto por cada módulo. Súbelo por tanto a una rama distinta que puedes llamar 72horas-<nombre_módulo>, y así separamos la discusión de cada uno. - Sobre lo que dices, efectivamente no es muy buena técnica meter directamente un ID, pero como la BD a migrar ya tiene esos datos así, no lo vamos a cambiar de momento. Quita entonces en esta rama ese otro módulo y ahora reviso el primero. Un saludo. P.D.: Acuérdate de ir poniendo las ramas como privadas. -- https://code.launchpad.net/~dani-ds/avanzosc/72horas/+merge/222913 Your team Avanzosc_security is requested to review the proposed merge of lp:~dani-ds/avanzosc/72horas into lp:~avanzosc-security-team/avanzosc/72horas. -- Mailing list: https://launchpad.net/~avanzosc Post to : [email protected] Unsubscribe : https://launchpad.net/~avanzosc More help : https://help.launchpad.net/ListHelp
