jnguiot
commented
3 months ago Est-ce que tu prévois une site de préprod comme test.collectives?
Je verrais. Maintenant que j'ai tout passé sur kubernetes, je pourrais très facilement faire un nouveau site. Je me demande même si on ne devrait pas faire ça pour Annecy...
Je regarde tes remarques et je les corrige :)
Cette pull request est utile pour le déploiemennt du site des collectives pour le paiement des formations au niveau du CD. N'ayant pas de compte extranet national, cette PR permet aux utilisateurs de s'inscrire par eux même, comme sur n'importe quel site.
Cette PR contient plusieurs éléments:
Design
C'est les commits de #710 : j'ai fait le travail à partir de cette branche. Rien de particulier
User Type
Pour différencier les comptes synchronisés avec l'extranet et les comptes locaux, j'ai remplacé l'argument
is_testpartype, ce qui permet d'avoir un type Extranet et un type Local. Par la suite, on pourra ajouter d'autres types, comme "Guide/BE" ou "Carte Découverte".Refactoring auth route
Rien de particulier, mais vu que le fichier
collectives/routes/auth.pycommencait à être trop long, je l'ai séparé en module. Il faudra surement le faire pourevent.pyaussiFlask config en DB
Pour utiliser recaptcha, j'avais besoin de mettre des paramètre dans app.config, mais je souhaitais pouvoir les changer à chaud. Donc j'ai créé une classe
DBAdaptedFlaskConfigqui hérite de la classeConfigde Flask, mais qui check les configurations en DB si celle demandée n'est pas dans config.py. Le fichier reste cependant prioritaire.Comptes locaux
L'objectif est de permettre l'utilisation du site sans l'extranet. Ainsi, une personne peut créer un compte local. A noter que celà veut dire que les données indiquées dans son profil ne sont pas vérifiées...