Skip to content

Nuevo campo de email de socio, obligatorio y único#372

Merged
euribates merged 2 commits intomainfrom
issue#371
Oct 6, 2021
Merged

Nuevo campo de email de socio, obligatorio y único#372
euribates merged 2 commits intomainfrom
issue#371

Conversation

@euribates
Copy link
Copy Markdown
Collaborator

Fixes #371

@jdevera
Copy link
Copy Markdown
Contributor

jdevera commented Sep 22, 2021

Cuál es ahora la semántica asociada a la relación entre member y user? Pregunto porque veo que conservamos ambos y además vienen relacionados.
Un member siempre es un user, pero un user no tiene que ser member, entiendo. ¿Qué es un user que no es member? ¿Por qué lo tenemos guardado?

Comment thread apps/members/migrations/0025_email_mandatory_and_unique.py
Comment thread apps/members/migrations/0024_copy_member_email_from_user_model.py
@sdelquin sdelquin removed their request for review September 22, 2021 21:06
@jdevera jdevera self-requested a review October 6, 2021 16:56
Copy link
Copy Markdown
Contributor

@jdevera jdevera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Algún test por ahí molaría :P

Pero dale.

@euribates euribates merged commit d696021 into main Oct 6, 2021
@euribates euribates deleted the issue#371 branch October 6, 2021 17:06
@euribates euribates added hacktoberfest Aptas para Hacktoberfest hacktoberfest-accepted HacktoberFest Accepted labels Oct 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest Aptas para Hacktoberfest hacktoberfest-accepted HacktoberFest Accepted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incluir campo email en el módelo Member

2 participants