Actualizar eventos meetup bot #197
Conversation
Al intentar abrirlo, aparece un error 500 de parte de meetup Ademas la información del mismo se enecuentra incompleta https://www.meetup.com/pythonbogota/events/238435129/
|
Adicione a blacklist el evento https://www.meetup.com/pythonbogota/events/238435129/ por que estaba roto, y no se deberia tener data invalida en el pr Scot3004#3 |
.github/workflows/update-events.yml
Outdated
|
|
||
| on: | ||
| schedule: | ||
| - cron: "0 12 * * 6" # Todos los sabados a las 12:00 |
There was a problem hiding this comment.
¿A qué se debe que sea cada sábado? Hasta el momento solo hay 2 comunidades activas y cada una tiene eventos mensuales
There was a problem hiding this comment.
Propongo que sea mensual y que sea entre semana para no recibir notificaciones los fines de semana jajaja. Puede ser el primer miercoles del mes o algo asi @Scot3004
There was a problem hiding this comment.
No me aparecia el comentario.
En cuanto al por que semanal los sabados es un efecto de copiar y pegar, en el de pybaq lo tenemos semanal los martes a las 12 😄 ten en cuenta que los eventos deberian actualizarse en 2 momentos, cuando se crea en meetup y cuando ya paso, entonces ese mismo evento en cada una de las 4 semanas apareceria de forma distinta
There was a problem hiding this comment.
Sera correcta esta expresión?
0 0 1,2,3,4,5,6,7 * 3
There was a problem hiding this comment.
Esta arroja una mas simple ¿qué opinas?
https://crontab.guru/#5_4_15__
There was a problem hiding this comment.
Acabo de ver que la pagina que pasaste da cuando seria la proxima ejecución y me suena mas el 6 a las 7pm :D
There was a problem hiding this comment.
acabo de caer en cuenta que no correria miercoles sino martes :)
There was a problem hiding this comment.
En ese caso si tocaria buscar una regla mas específica
There was a problem hiding this comment.
Que tal esta, el primer miercoles del mes a medio dia?
There was a problem hiding this comment.
Lo que no me cuadra es que diga que que se ejecuta en 7 días diferentes y además el miércoles (1 and 2 and 3 and 4... and Wednesday).
Scot3004
force-pushed
the
actualizar-eventos-script
branch
from
5cef979 to
23f0409
Compare
| @@ -0,0 +1,63 @@ | |||
| name: "Actualizar eventos meetup" | |||
|
|
|||
| on: | |||
There was a problem hiding this comment.
@Scot3004 puedes probar con este cambio para ejecutar el job antes de mezclarlo?
O lo probaste en algun otro repositorio propio?
| on: | |
| on: | |
| pull_request: | |
| branches: | |
| - develop |
There was a problem hiding this comment.
No lo quise poner, la idea es que la actualización de eventos se haga bajo una de estas 2 premisas
- Una vez al mes para actualizar según lo que este en meetup, para que no se nos quede tan atras como ahora que se queda un año y nadie envia un pr actualizando eventos
- A demanda, por ejemplo si quiero promocionar un evento en mi grupo local y lo acabo de crear, con este dispatch puedo forzar que se actualicen los eventos
En cuanto al por que no considero conveniente que se corra cuando se hace un pull request, cada vez que se haga un pull request se creara otro con los eventos actualizados, a decir verdad para eso tenemos deploy.yml
En si este job solo sirve para actualizar los eventos es algo así como una etl que extrae la data de meetup, la transforma en content.lr y luego lo carga en github
There was a problem hiding this comment.
Aca está el ejemplo de como deberia funcionar
https://github.com/PyBAQ/website/actions/workflows/actualizar-eventos-meetup.yml
Ten en cuenta que tiene ciertas diferencias, pero en esencia hacen lo mismo
Actualización de eventos sabados
Scot3004#1
https://github.com/Scot3004/sitio-web/actions/runs/9472096159