Closed nabiu256 closed 2 years ago
El package consta bàsicament d'un fitxer on s'implementa la lògica (aquí) i un fitxer per configurar-lo com a app de Django (aquest). És un total de menys de 100 línies de codi que poden ser ben simplificades sense problema.
Pros de copiar el codi:
Cons:
poetry update
i tenir-lo en l'última versió, sense haver de fer res més, en canvi així s'haurà de comprovar: 1. mirar que estigui actualitzat, 2. copiar el codi i enganxar i comprovar que tot funciona. Altra vegada però, crec que és altament improvable que mai haguem de fer això.Dit això, jo soc partidària del que valoris tu @perepicornell . Si eliminar dependències i deixar-ho més net és algo que prefereixes, ho puc implementar en un moment.
Moltes gràcies per fer aquest resum, super útil ^^ Doncs de moment voto per deixar-lo, i quan ens trobem amb que en una actualització de Python o Django deixa de ser compatible, ja el copiarem i adaptem nosaltres.
El dc., 18 de maig 2022, 17:12, JanaL @.***> va escriure:
El package consta bàsicament d'un fitxer on s'implementa la lògica (aquí https://github.com/CleitonDeLima/django-login-required-middleware/blob/master/login_required/middleware.py) i un fitxer per configurar-lo com a app de Django (aquest https://github.com/CleitonDeLima/django-login-required-middleware/blob/master/login_required/apps.py). És un total de menys de 100 línies de codi que poden ser ben simplificades sense problema.
Pros de copiar el codi:
- Ens estalviem tenir una dependència més en el pyproject
- Podem fins i tot minimitzar el codi i treure un parell de funcionalitats que té. Pel que nosaltres volem fer servir, amb un terç d'aquest codi ja seria suficient.
- Es pot seguir actualitzant el codi a mesura que s'actualitzi la repo simplement copiant i enganxant codi.
- El funcionament és molt senzill i no té cap mena de pinta que li haguem de fer cap mena d'actualització, així que el codi que copiem i enganxem ara potser ens serveix ja per sempre, o com a mínim per molt de temps.
Cons:
- Tenint-lo com a dependència és tan senzill com fer un poetry update i tenir-lo en l'última versió, sense haver de fer res més, en canvi així s'haurà de comprovar: 1. mirar que estigui actualitzat, 2. copiar el codi i enganxar i comprovar que tot funciona. Altra vegada però, crec que és altament improvable que mai haguem de fer això.
- Al copiar i enganxar codi (i així fer-li una petita modificació), s'haurà de comprovar que no introduïm cap bug (tot i que altra vegada, és una dependència petitíssima, no hauria de portar problemes).
Dit això, jo soc partidària del que valoris tu @perepicornell https://github.com/perepicornell . Si eliminar dependències i deixar-ho més net és algo que prefereixes, ho puc implementar en un moment.
— Reply to this email directly, view it on GitHub https://github.com/codicoop/boilerplate_django/issues/34#issuecomment-1130144223, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABFVRQTJZDWQTYNMD5XA3U3VKUCG3ANCNFSM5WIY3GGA . You are receiving this because you were mentioned.Message ID: @.***>
Perfecte. Tanco issue doncs.
Valorar si el package django-login-required-middleware és tan senzill com perquè tingui més sentit copiar el codi cap a un fitxer a /base i així eliminar una dependència... (molaria deixar una menció al propi codi o al readme indicant que n'hem fet ús).