ctrl-alt-d / django-aula

Gestió de presencia, incidències i més en centres educatius i acadèmies.
Other
18 stars 28 forks source link

Docker has been added #172

Closed jbagot closed 2 years ago

jbagot commented 2 years ago

I added docker with postgresql. It is working but there are some scripts that uses sqlite that will not work. I changed dev settings to use env vars, we can improve that much more.

170 solved

jbagot commented 2 years ago

D'acord ara faré que el settings pugui anar amb sqlite, però ja que tens postgres en el docker-compose jo el que faria seria començar a treure el sqlite del projecte i sempre utilitzar postgres per dev també. Però això ja és cosa teva. el psycopg2-binary li he deixat perquè sinó serà un problema per qui vulgui fer servir postgres. Potser hauríem de pensar millor com fer la gestió de DBs si és que s'ha de donar suport a varies. És així? S'ha de donar suport a varies DBs o quin és l'objectiu?

ctrl-alt-d commented 2 years ago

S'ha de donar suport a varies DBs o quin és l'objectiu?

Eliminar barreres d'entrada. Nosaltres ho tenim molt interioritzat com aixecar els Dockers, esborrar-los, etc però no tinc clar que tothom que col·labora al projecte ho sàpiga fer.

Entenc perfectament el teu punt de vista. Amb el canvi que demano tindríem els dos entorns fàcilment? Amb i sense Dockers? Respon una mica això al que em preguntes?

jbagot commented 2 years ago

Si, si poso el sqlite per defecte si no es defineix un .env.db sí. Instal·laré django-environ per gestionar les variables d'entorn que ho facilita molt més i pots posar valors per defecte i dir de quin tipus són. És un altre paquet però crec que és important IMO

ctrl-alt-d commented 2 years ago

Guai, em sembla bé. Al codi que et posava ja hi havia valors per defecte amb el os.getenv però bona idea i més net amb django-environ com a Heroku :+1:

jbagot commented 2 years ago

Ja ho he fet, revisa-ho i em dius :)