Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parametrizar datos del cron para los avisos previo a las clases #36

Closed
ilitteri opened this issue Mar 24, 2022 · 4 comments
Closed

Parametrizar datos del cron para los avisos previo a las clases #36

ilitteri opened this issue Mar 24, 2022 · 4 comments
Labels
enhancement New feature or request

Comments

@ilitteri
Copy link
Member

Actualmente se tienen que modificar a mano cuatrimestre a cuatrimestre. El código está en Client.ts.

  • Una forma seria que chequee todo el tiempo en que parte del año estamos.
  • Otra sería tener una variable de entorno (que deberíamos modificar cuatrimestre a cuatrimestre en heroku).

Se aceptan otras propuestas.

@josuebouchard
Copy link
Collaborator

Me gusta mas la segunda opcion. Es un trabajo que solo hay que hacer una vez por cuatrimestre (ya hay que verificar las variables de entorno una vez por cuatrimestre de todas maneras), y que permite que si fuera necesario, podriamos cambiarlo a mitad de cuatrimestre sin tener que hacer otro PR. Ademas es mas simple de implementar.

@ilitteri ilitteri added the enhancement New feature or request label Mar 24, 2022
@ilitteri
Copy link
Member Author

Me gusta mas la segunda opcion. Es un trabajo que solo hay que hacer una vez por cuatrimestre (ya hay que verificar las variables de entorno una vez por cuatrimestre de todas maneras), y que permite que si fuera necesario, podriamos cambiarlo a mitad de cuatrimestre sin tener que hacer otro PR. Ademas es mas simple de implementar.

Tal cual! Esto que comentás me hizo pensar en que estaría bueno dejar escrito qué variables de entorno cambian cada cuatrimestre, por qué, y cómo hacerlo (para esta última agregar la limitación de heroku en la cantidad de usuarios que pueden modificar el proyecto)

@josuebouchard
Copy link
Collaborator

Solo para que quede registro, lo del tratamiento de las variables de entorno, actualizacion y documentacion se va a tratar en #37.
Este issue solo queda para lo relativo a la parametrizacion de los cron jobs.

@ilitteri
Copy link
Member Author

@IAvecilla pensó que sería mejor dejar todos los meses en dónde se van a dictar clases, lo cuál tiene todo el sentido del mundo. Se va a abrir un issue escífico para eso, dejando a este sin sentido.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants