search

okfnfr / dada-france-theme

Le theme alaveteli utilisé par Ma Dada
https://madada.fr
MIT License
4 stars 3 forks source link

Ajouter explication sur la Version Pro dans le Readme #101

Open pzwsk opened 3 years ago

pzwsk commented 3 years ago

Bonjour @laurentS

pourrais tu mettre à jour le Readme en indiquant quels sont les versions que l'on utilise pour la version courante et la version Pro, ainsi que les liens vers les répertoires de ces versions ?

Et si besoin, supprimer les répertoires non utilisés dans notre organisation.

Merci

laurentS commented 3 years ago

@pzwsk pas tout à fait sûr de ce que tu entends par là. Je viens de relire le fichier, il me semble à jour. La version pro utilise exactement les mêmes branches qu'avant, une par serveur, on a simplement modifié la config (et je ne vais pas recopier la doc de mysociety).

pzwsk commented 3 years ago

Pour préciser : ma compréhension est que avant la version Pro, le répertoire dada-france-theme de okfnfr venait remplacer les fichiers du répertoire alaveteli de okfnfr.

Questions :

Sans ces informations, il est difficile de savoir où regarder pour les modifications à effectuer et fichiers à ajouter dans dada-france-theme.

Merci beaucoup, Pierre

laurentS commented 3 years ago

Pour préciser : ma compréhension est que avant la version Pro, le répertoire dada-france-theme de okfnfr venait remplacer les fichiers du répertoire alaveteli de okfnfr.

Ok, je comprends mieux. Je vais supprimer le fork dans ce cas (une fois que tu m'auras confirmé que tu n'as besoin de rien dedans, tu sembles être le dernier à y avoir modifié des fichiers). Il n'a servi que quelques semaines, le temps pour mySociety de merger une petite correction que j'avais faite dans le code.

* a quoi sert désormais le répertoire `alaveteli` de okfnfr ? Je vois que celui n'a pas été mis à jour donc j'imagine que ce n'est pas celui qu'on utilise.

Il ne sert plus, effectivement. Si on a besoin un jour de corriger un bug, on pourra toujours reforker, c'est comme ça que marche github :)

* quels sont les versions `alaveteli` et `alaveteli pro `que l'on utilise ?

Tout ça est défini dans dada-core, précisement ici. Note que la notion de "pro" correspond plus à l'activation d'une partie des fonctions du même code qu'à une version différente du logiciel. Si tu penses au texte du modèle de demande, par exemple, il est dans transifex, sous 2 clés dont le nom n'indique pas du tout cette distinction. Voir le README pour quelques tuyaux.

Sans ces informations, il est difficile de savoir où regarder pour les modifications à effectuer et fichiers à ajouter dans dada-france-theme.

Oui. J'ai fait 2-3 mises à jour dans les README. Cela dit, même avec ces infos, je trouve que c'est compliqué de savoir où modifier quoi... Jusqu'ici, j'ai presque toujours cherché le texte qui m'intéressait avec un bon vieux grep, combiné avec les devtools de firefox pour trouver l'id de l'élement html que je veux modifier.

Dis moi si le README te semble plus clair, et sinon, n'hésites pas à me poser plus de questions, et j'y répondrai dans la doc, histoire de garder une trace.