avanzosc team mailing list archive
-
avanzosc team
-
Mailing list archive
-
Message #00449
Re: [Merge] lp:~dani-ds/avanzosc/72horas into lp:~avanzosc-security-team/avanzosc/72horas
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.
References