Creado modelo Talk #209

Scot3004 · 2023-08-13T00:39:58Z

Work in progress
Se creo el modelo para charla, lenguaje y nivel de la charla
Está pendiente la creación de los escenarios de prueba y los respectivos tests

pyjavo · 2023-08-22T18:57:50Z

temii/talks/models.py

+from temii.users.models import User
+
+
+class TalkLanguage(models.Model):


Si solo van a haber 2 opciones de lenguaje de la charla (español e inglés). Sugiero colocar las opciones en el mismo modelo Talk y no crear una tabla nueva en la base de datos solo para estos 2 valores.

Ejemplo
https://docs.djangoproject.com/en/4.2/ref/models/fields/#choices

from django.db import models class Student(models.Model): FRESHMAN = "FR" SOPHOMORE = "SO" JUNIOR = "JR" SENIOR = "SR" GRADUATE = "GR" YEAR_IN_SCHOOL_CHOICES = [ (FRESHMAN, "Freshman"), (SOPHOMORE, "Sophomore"), (JUNIOR, "Junior"), (SENIOR, "Senior"), (GRADUATE, "Graduate"), ] year_in_school = models.CharField( max_length=2, choices=YEAR_IN_SCHOOL_CHOICES, default=FRESHMAN, ) def is_upperclass(self): return self.year_in_school in {self.JUNIOR, self.SENIOR}

pyjavo · 2023-08-22T18:58:16Z

temii/talks/models.py

+        return self.name
+
+
+class TalkLevel(models.Model):


Creo que solo hay 3 niveles:

Beginner

Intermediate

Advance
También creo que aplica usar choices (o TextChoices)

Ejemplo
https://docs.djangoproject.com/en/4.2/ref/models/fields/#choices

Te refieres a algo como esto?

LEVEL_BEGINNER = 1 LEVEL_INTERMEDIATE = 2 LEVEL_AVANCED = 3 LEVEL_CHOICES = [ (LEVEL_BEGINNER, "Principiante"), (LEVEL_INTERMEDIATE, "Intermedio"), (LEVEL_AVANCED, "Avanzado"), ]

pyjavo

¡Excelente! 💪💪💪 Estamos cada vez más cerca. Este PR está relacionado con el issue #140

Unas sugerencias:

Falta el campo de Mes(es) de la charla para que el ponente sugiera su fecha preferida para hablar (ver formulario de Google como guía).
Falta el campo modalidad de la charla para saber si la charla será "Presencial en Barranquilla" o "Virtual (estoy fuera de Barranquilla)"
Resetear las migraciones del Modelo Talk para que solo exista 0001_initial.py en este PR
Tal vez conviene ver https://simpleisbetterthancomplex.com/tutorial/2016/07/26/how-to-reset-migrations.html

Scot3004 · 2023-09-19T21:06:21Z

Para los meses creo que sería mucho mejor postular a eventos, (que en este contexto sería lo mismo que meses)

pyjavo · 2023-09-19T22:28:56Z

@Scot3004 Podrías explicarme, ¿a qué te refieres con postular a eventos?

Scot3004 · 2023-09-19T22:35:16Z

Es que aparezca un listado de eventos y puedas escoger en que evento te gustaría que aparezca
En vez de asociarlo al mes a los posibles eventos

pyjavo · 2023-10-05T03:00:11Z

@Scot3004 pero creo que cuando el ponente aplica, aún no hay un evento o eventos disponibles. Esa persona solo sabe de antemano que mes o meses de su agenda personal puede disponer para su propuesta de charla.

Pienso que el paso a paso inicial es el siguiente (disculpa que no es un diagrama de flujo):

Persona aplica con una nueva charla
La charla entra a la lista de posibles charlas
El admin escoge una o varias charlas para un futuro evento
Se crea el evento con fecha, hora, lugar y charla(s) elegida(s).
Se promociona el evento en redes sociales.
Una vez terminada la charla, se recopilan sus recursos para publicar en el sitio web. Ej: URL de youtube, diapositivas, URL del repositorio o se sube a PyBAQ/Notebooks. Todo dependiendo de cada caso.

¿Qué opinas?

Scot3004 · 2023-10-05T13:57:11Z

Ok, ¿Que tipo de datos le ponemos a los meses?, CharField de 300?

pyjavo · 2023-10-06T15:06:38Z

@Scot3004 si vamos a guardar caracteres de texto con los nombres de los meses, sea "enero", "febrero", "marzo", etc, entonces, según mis cálculos, septiembre es el mes con mayor número de caracteres. Tiene 10 caracteres (le siguen "noviembre" y "diciembre" con 9).
Podríamos hacer un Charfield(20) para dejar un buffer de 10 caracteres de más, solo por si acaso jeje.

¿Qué opinas?

Scot3004 · 2023-10-06T16:04:29Z

Al menos 100 pa que pueda meter 9 meses (más o menos si los coloca en formato largo separado con espacios)

pyjavo · 2023-10-06T22:22:48Z

@Scot3004 tienes razón. No había caído en cuenta de ese escenario

Scot3004 · 2023-10-07T18:33:20Z

temii/talks/models.py

+        ES = "es", _("Spanish")
+        EN = "en", _("English")
+
+    class InPerson(models.TextChoices):


Deberia ser IntegerChoices

- Adicionados campos meses y presencialidad - Actualizado el archivo de traducción - Migraciones para talks creadas desde 0

Scot3004 · 2023-10-07T18:40:44Z

Listo, Por alli veo unos inconvenientes con la localización, no se por que suceden, pero creo que ya este PR esta lo suficiente grande, y es mas cuestion de forma que de fondo

Scot3004 · 2023-10-07T18:47:02Z

Encontre la causa python manage.py compilemessages nunca se llama, mando eso dentro de este PR?

Scot3004 · 2023-10-07T18:48:19Z

Scot3004 · 2023-10-08T21:36:57Z

Ya se aplicaron los 3 puntos
Se creo un campo meses con capacidad de 100 caracteres
La modalidad de la charla se aborda en el campo "Presencialidad" que tiene 2 opciones "En sitio" y "Virtual"
Presencialidad en Sitio, es para cuando la charla va a ser presencial en el lugar del evento, igual el hecho que sea virtual es tambien estar presente en la reunión solo que de forma virtual
Los flujos de charlas, lo abordaria mejor en otro issue, la verdad ya este esta lo suficientemente grande, y eso que la gestión de la charla se hace en el admin
No se abordo la creación del formulario de charla en el front office, para permitir que eso se aborde en un ticket futuro, solo lo que si o si requeria tener en el modelo

pyjavo

De acuerdo. Lo restante lo podemos abordar en otro PR. ¡Buen trabajo!

Scot3004 marked this pull request as draft August 13, 2023 00:40

pyjavo reviewed Aug 22, 2023

View reviewed changes

Scot3004 added 2 commits September 5, 2023 21:16

Creado modelo Talk

dd35100

Enums Choices en los talks

190f988

Scot3004 force-pushed the talks branch from ef0c1d1 to 190f988 Compare September 6, 2023 03:44

Talks tests

b657cb9

Scot3004 force-pushed the talks branch from 3535c3d to b657cb9 Compare September 6, 2023 04:01

Español en charlas

5895cf6

Scot3004 marked this pull request as ready for review September 6, 2023 04:44

Eliminadas traducciones innecesarias

15da893

pyjavo requested changes Sep 13, 2023

View reviewed changes

Merge branch 'PyBAQ:master' into talks

bffbafd

Scot3004 force-pushed the talks branch 4 times, most recently from 33e8f6c to 413e83e Compare October 7, 2023 18:28

Scot3004 commented Oct 7, 2023

View reviewed changes

Actualizado modelo talks

0e1e98f

- Adicionados campos meses y presencialidad - Actualizado el archivo de traducción - Migraciones para talks creadas desde 0

Scot3004 force-pushed the talks branch from 413e83e to 0e1e98f Compare October 7, 2023 18:39

pyjavo approved these changes Oct 10, 2023

View reviewed changes

pyjavo merged commit 5c255a2 into PyBAQ:master Oct 10, 2023
2 checks passed

pyjavo mentioned this pull request Nov 24, 2023

Postular charla #140

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Creado modelo Talk #209

Creado modelo Talk #209

Scot3004 commented Aug 13, 2023

pyjavo Aug 22, 2023

pyjavo Aug 22, 2023 •

edited

Loading

Scot3004 Aug 23, 2023 •

edited

Loading

pyjavo Aug 23, 2023

pyjavo left a comment •

edited

Loading

Scot3004 commented Sep 19, 2023

pyjavo commented Sep 19, 2023

Scot3004 commented Sep 19, 2023

pyjavo commented Oct 5, 2023

Scot3004 commented Oct 5, 2023

pyjavo commented Oct 6, 2023

Scot3004 commented Oct 6, 2023

pyjavo commented Oct 6, 2023

Scot3004 Oct 7, 2023

Scot3004 commented Oct 7, 2023

Scot3004 commented Oct 7, 2023 •

edited

Loading

Scot3004 commented Oct 7, 2023

Scot3004 commented Oct 8, 2023

pyjavo left a comment

		from temii.users.models import User


		class TalkLanguage(models.Model):

Creado modelo Talk #209

Creado modelo Talk #209

Conversation

Scot3004 commented Aug 13, 2023

pyjavo Aug 22, 2023

Choose a reason for hiding this comment

pyjavo Aug 22, 2023 • edited Loading

Choose a reason for hiding this comment

Scot3004 Aug 23, 2023 • edited Loading

Choose a reason for hiding this comment

pyjavo Aug 23, 2023

Choose a reason for hiding this comment

pyjavo left a comment • edited Loading

Choose a reason for hiding this comment

Scot3004 commented Sep 19, 2023

pyjavo commented Sep 19, 2023

Scot3004 commented Sep 19, 2023

pyjavo commented Oct 5, 2023

Scot3004 commented Oct 5, 2023

pyjavo commented Oct 6, 2023

Scot3004 commented Oct 6, 2023

pyjavo commented Oct 6, 2023

Scot3004 Oct 7, 2023

Choose a reason for hiding this comment

Scot3004 commented Oct 7, 2023

Scot3004 commented Oct 7, 2023 • edited Loading

Scot3004 commented Oct 7, 2023

Scot3004 commented Oct 8, 2023

pyjavo left a comment

Choose a reason for hiding this comment

pyjavo Aug 22, 2023 •

edited

Loading

Scot3004 Aug 23, 2023 •

edited

Loading

pyjavo left a comment •

edited

Loading

Scot3004 commented Oct 7, 2023 •

edited

Loading