pyarweb icon indicating copy to clipboard operation
pyarweb copied to clipboard

problemas de estilo en página de confirmación de email

Open cmdelatorre opened this issue 7 years ago • 3 comments

La página http://localhost:8000/accounts/confirm-email/xxxxxx a la que te manda cuando confirmas tu email (depsués de registrarte) no tiene estilos screenshot from 2018-08-04 14-22-18

cmdelatorre avatar Aug 04 '18 17:08 cmdelatorre

Cree el PR #447 para solucionar este bug. Rompieron los tests pero no entiendo bien por qué, en mi entorno local con make test anda de maravilla

FedericoJurio avatar Apr 03 '19 21:04 FedericoJurio

Cree el PR #447 para solucionar este bug. Rompieron los tests pero no entiendo bien por qué, en mi entorno local con make test anda de maravilla

Los tests pasan con python3.6. Fijate si podes correrlo con python3.5 porque travis falla con esa version en el test test_recent_is_a_list_with_at_most_10_items:

======================================================================
ERROR: test_recent_is_a_list_with_at_most_10_items (community.tests.test_homepage.GetContextDataRecentKeyTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/virtualenv/python3.5.6/lib/python3.5/site-packages/django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)
psycopg2.IntegrityError: duplicate key value violates unique constraint "auth_user_username_key"
DETAIL:  Key (username)=(jonesjohn) already exists.

SIno la otra es skippear esa version en travis. Pero no se en que version de python estamos corriendo la web.

eduzen avatar Apr 03 '19 22:04 eduzen

Gracias @eduzen, me puse a ver el log del build despues de hacer el comentario :stuck_out_tongue: . El error es claro, se quiso insertar en la base de datos el username jonesjohn ya existiendo un registro con el mismo username en la tabla. Asumo - porque no tengo ninguna evidencia concreta - que alguno de los tres factories que están en test_recent_is_a_list_with_at_most_10_items (JobFactory, EventFactory, o NewsArticleFactory) fue el responsable de crear un usuario duplicado.

Hice el build en forma local con Python 3.5.7 y funcionó sin problemas, así que volví a triggerear el build en Travis y salió andando.

El PR está listo para ser revisado!

FedericoJurio avatar Apr 04 '19 01:04 FedericoJurio