#29 Add conf file and fix 2 issues

Fusionné
jocelyn a fusionné 11 commits à partir de ljf/master vers FFDN/master il y a 8 ans
ljf a commenté il y a 8 ans

Hi, Here is my contribution to wifi-with-me :) .

The most important is the creation of several conf var. Thanks to these, everyone in french country could create his own wireless isp. In parallel, I created a YunoHost package on this repo. https://code.ffdn.org/ljf/wifiwithme_ynh

The 2 other commit fix a missing dependencies and optimize the logging conf to be able to be compatible with gunicorn.

See u, ljf

#19 #28

Hi, Here is my contribution to wifi-with-me :) . The most important is the creation of several conf var. Thanks to these, everyone in french country could create his own wireless isp. In parallel, I created a YunoHost package on this repo. https://code.ffdn.org/ljf/wifiwithme_ynh The 2 other commit fix a missing dependencies and optimize the logging conf to be able to be compatible with gunicorn. See u, ljf #19 #28
jocelyn a commenté il y a 8 ans
Propriétaire

Thanks :-)

Currently reviewing…

Out of curiosity : why is sqlparse required ?

Thanks :-) Currently reviewing… Out of curiosity : why is sqlparse required ?
jocelyn a commenté il y a 8 ans
Propriétaire

Thanks a lot ! That's good ; some things to fix/clarify before the merge IMHO:

  • ced61b7d3c : I do not understand the intent of the commit
  • d2109f2fa7 : could you create a dedicated template tag (something called like obfuscate_email) instead of that too generic replace ?

PS: Well spotted for the legal page and the connect_local / connect_internet stuff :-) (that new Django version is not yet in use at FAImaison…)

Thanks a lot ! That's good ; some things to fix/clarify before the merge IMHO: - ced61b7d3c : I do not understand the intent of the commit - d2109f2fa7 : could you create a dedicated template tag (something called like `obfuscate_email`) instead of that too generic `replace` ? PS: Well spotted for the legal page and the connect_local / connect_internet stuff :-) (that new Django version is not yet in use at FAImaison…)
ljf a commenté il y a 8 ans
Publier

Thanks for reviewing !

I don't know why sqlparse is in dependencie, but on a clear setup the python manage.py command ask me to install sqlparse. You are using an sqlite db may be it's an explanation ?

  • ced61b7d3c : without this bugfix, latitude and longitude are written in a french format number but coma are not javascript compatible so the latitude and longitude was truncated.
  • d2109f2fa7 : if I found the time yes, I am agree with this idea.
Thanks for reviewing ! I don't know why sqlparse is in dependencie, but on a clear setup the python manage.py command ask me to install sqlparse. You are using an sqlite db may be it's an explanation ? * ced61b7d3c : without this bugfix, latitude and longitude are written in a french format number but coma are not javascript compatible so the latitude and longitude was truncated. * d2109f2fa7 : if I found the time yes, I am agree with this idea.
jocelyn a commenté il y a 8 ans
Propriétaire

Got it for sqlparse.

Waiting for you fix on d2109f2fa7 if it's ok for you :-).

Got it for sqlparse. Waiting for you fix on d2109f2fa7 if it's ok for you :-).
ljf a commenté il y a 8 ans
Publier

Done

Done
jocelyn a commenté il y a 8 ans
Propriétaire

@ljf Awesome! Thanks for your attention to details and shaping things properly.

merge d2109f2fa7 and d2109f2fa7 and we're good for merge.

@ljf Awesome! Thanks for your attention to details and shaping things properly. merge d2109f2fa7 and d2109f2fa7 and we're good for merge.
ljf a commenté il y a 8 ans
Publier

I have merged the 2 commit about email obfuscation

I have merged the 2 commit about email obfuscation
Cette Pull Request a été fusionnée avec succès !
Connectez-vous pour rejoindre cette conversation.
Aucun jalon
Pas d'assignataire
2 Participants
Chargement…
Annuler
Enregistrer
Il n'existe pas encore de contenu.