Moderadores para ofertas de trabajo ref #323

[btenaglia]

No description provided.

[angvp]

@btenaglia Estuve revisando el PR, primero que nada muchas gracias por el laburón de tomar este issue y hacerlo. Por ahí agregaría un par de cosas:

• Unifica las migraciones (hay varias migrations que los cambios son mínimos), si podés squashearlas y enviar las migraciones squasheadas sería genial https://docs.djangoproject.com/en/1.8/topics/migrations/#squashing-migrations
• En vez de usar un fixture usa un data migration https://docs.djangoproject.com/en/1.8/topics/migrations/#data-migrations

Añadiría que agregaras un par de tests case en algunos de los filters que hiciste para irlos teniendo, pero esto quizás lo podríamos obviar ahora mismo y luego podemos hacer los tests de todo lo que está corriendo no testeado en otro PR.

Pero de resto como dije al principio, muy buen trabajo, si haces estos 2 cambios ya le hago merge.

Abrazo!

[angvp]

Hola @btenaglia !

Gracias por seguir metiéndole energía a esto, te comento 2 cosas:

• Cuando te comenté lo de squashear las migrations la idea era que squashearas las migrations que tu agregaste en este PR no todas las migrations pre existentes, yo revertiría este commit el 658a165 y haría el squash de nuevo, ya que estás en tu branch y para que quede prolijo, puedes hacer un rebase -i HEAD~2 (volver a la historia antes de este commit) y squashear solo esas migraciones.
• El build falló porque recién con @gilgamezh agregamos soporte para TravisCI .. si querés, luego de que termines, hace un pull rebase de develop a tu branch y así corre con los cambios.

[btenaglia]

@angvp @mgaitan alguna novedade del code review?.
Yo creo que ya estaría para el merge, el tema de travis está fallando por mas que hice merge de develop, pero el error es por los tests.

[angvp]

Merged

[mgaitan]

Gente querida, yo estoy muy colgado con la web, mis horas "libres" se han achicado gozosamente desde la llegada de mi hija. Pero soy conciente de que Bruno ha sido quien más ha laburado y esta tarea la hizo hace bocha y no está deployada. no podemos colgarnos tanto con la gente con pilas en colaborar.

hay algo a tener en cuenta para desplegarlo en producción?

---------- Forwarded message ----------
From: Bruno Tenaglia brunomartintenaglia@gmail.com
Date: 2015-11-23 14:30 GMT-03:00
Subject: Re: [pyar] [off topic] spam en trabajos.
To: Python Argentina pyar@python.org.ar

Hola, si yo lo noté hoy, están spameando mal.
Yo ya hice la funcionalidad para moderar los anuncios, falta implementarla nomás.

Sldos!

[angvp]

@gilgamezh cuando tengas tiempo hacemos un deploy, avísame por irc para coordinar hora, puede ser mañana a cualquier hora?