← Back to team overview

avanzosc team mailing list archive

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