Nouveau site du COF #551
No reviewers
Labels
No labels
devtype -- backend
devtype -- docs
devtype -- frontend
devtype -- user interface
difficulty -- easy
difficulty -- hard
difficulty -- normal
Doing
domain -- bda
domain -- bds
domain -- cof
domain -- core
domain -- kfet
Good first issue
priority -- high
priority -- low
priority -- medium
priority -- staff-wanted
status -- development
status -- discussion
status -- need review
status -- production
status -- ready to merge
status -- todo
To Do
type -- bug
type -- hygiene
type -- improvement
type -- new feature
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: DGNum/gestioCOF#551
Loading…
Reference in a new issue
No description provided.
Delete branch "evarin/site-cof"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Un tout nouveau site pour le COF
TODO restant :
added 1 commit
0e19abb5
- Cleaner homepageCompare with previous version
added 1 commit
5a22b1cd
- Affichage des actusCompare with previous version
unmarked as a Work In Progress
changed the description
changed the description
added 1 commit
adf43889
- Fixtures site cofCompare with previous version
added 113 commits
master
ea495e8f
- Archives beta8551ffcf
- Merge branch 'master' into evarin/site-cofCompare with previous version
added 1 commit
8488beeb
- Un seul modèle pour les actusCompare with previous version
marked the checklist item Mieux gérer Évènement / Actualités simples (notamment QuerySet) as completed
added 2 commits
c11ccf2e
- Tri des annuairesf8952225
- Apparence et ResponsivenessCompare with previous version
added 2 commits
ba870446
- Calendrier dynamique + sympa + jolies dates88e911ff
- Fixtures mises à jourCompare with previous version
added 1 commit
59116f2d
- International wagtail urlsCompare with previous version
marked the checklist item Calendrier dynamique (-> faire une API pour les events ? ou juste une view JSON à la main) as completed
marked the checklist item Affichage des news ? as completed
marked the checklist item CSS responsif as completed
marked the checklist item Pagination as completed
marked the checklist item I18N et switch EN/FR as completed
marked the checklist item Page et pseudo-captcha listes mails as completed
added 1 commit
63ce694b
- Fix dependencyCompare with previous version
added 1 commit
6d6c9955
- Fixtures fonctionnellesCompare with previous version
TODO : changer le fonctionnement de la page "listes mails" pour faire comme en K-Fêt un captchablock ?
(Actuellement erreur liée à l'i18n)
added 1 commit
34749760
- Revert urls prefixingCompare with previous version
added 1 commit
3d091f50
- Suppr captcha page, ajout block iframeCompare with previous version
marked the checklist item Couleurs (ping Erkan) as completed
Finalement, lien vers page élèves et block iframe pour les pages qui en ont besoin (aka demande petit cours) => tout est prêt
Pas besoin du
u
en python3, les chaînes de caractères sont unicode par défautOn pourrait peut-être donner un nom plus informatif à ce fichier
On pourrait peut-être donner un nom plus informatif à ce fichier
Idem, voire même fusionner les 3 fichiers en fait
Les parenthèses ne servent à rien
en python3 on peut juste écrire
super().get_context(request)
Qu'est ce qui se passe si page est un entier invalide ?
On pourrait utiliser les validators de django pour vérifier que page est bien un entier et prendre une valeur par défaut si ça fail
Ça éliminerait un
try … except
Par curiosité, ça veut dire quoi chapo ?
Chapô (Journalisme) Texte court coiffant un article, généralement typographié en gras, pour amener le lecteur à entrer dans l’article.
Hmm les validators c'est pour les forms surtout (et il n'y a pas de IntValidator). Le
try...except
est intéressant pour le EmptyPage (i.e. accès à un numéro de page trop grand), qui ne peut être détecté que par le paginator. Donc autant tout gérer dans le mêmetry...except
non ?(En soi je crois que j'avais juste copié le code de https://simpleisbetterthancomplex.com/tutorial/2016/08/03/how-to-paginate-with-django.html )
changed this line in version 14 of the diff
changed this line in version 14 of the diff
changed this line in version 14 of the diff
changed this line in version 14 of the diff
changed this line in version 14 of the diff
changed this line in version 14 of the diff
added 1 commit
6d72644e
- Clean code upCompare with previous version
ok ça marche
resolved all discussions
que signifie le
debugged_
ici ?Là on veut peut-être utiliser un format différent en fonction de la langue.
Et je pense qu'on devrait aussi stocker ce format (
"d F Y"
) danscof.locale.fr
etcof.locale.en
C'est un peu compliqué toutes ces boucles. Il me semble que ceci fait la répartition en jours de façon un poil plus claire :
Le
curday
dans ton code correspond au champday
iciC'est un peu compliqué toutes ces boucles. Il me semble que ceci fait la répartition en jours de façon un poil plus claire :
Le
curday
dans ton code correspond au champday
icipourquoi pas simplement
str(next_month.month)
etcMême remarque que plus tôt dans les templates : est-ce que ça ne devrait pas plutôt être dans
cof/locale/{fr,en}/formats
et changer en fonction de la langue ?added 1 commit
6d6ba70b
- CSS++Compare with previous version
added 1 commit
954a6fdb
- Wagtail requirement updateCompare with previous version
👋 @champeno :-)
Ça en est où ? T'as le temps de t'occuper des commentaires de Martin ?
Ah euh non j'avais oublié qu'il restait des trucs à régler ici. J'avais surtout en tête la question des formats d'urls globaux qui bloquaient la fusion du site, je sais pas ce qui avait été décidé finalement.
Je peux m'en occuper demain je pense.
added ~62 ~60 ~26 labels
Si je me souviens bien : ModelTranslation override le système des @route, ce qui fait que COFUtilPage ne doit pas être une page traduite pour fonctionner. Mais alors
get_absolute_url
se met à planter (à cause de problèmes d'héritage je crois) et le seul moyen que j'ai trouvé était de faire une autre fonction qui prenne request en argument (donc pas un override deget_absolute_url
). Il faudrait que je voie si ces problèmes ont été résolus dans les màj de wagtail et modeltranslationEn fait il suffit de faire
|date
tout seul et il utilise les formats par défaut définis (ou pas) dans les settings.Hummm je peux faire ça mais dans tous les cas je me retrouve à faire les boucles après pour remplir les cases, et ça ne m'avance pas tant (la boucle est obligatoire pour gérer les (rares) cas d'overlaps du genre un event le 2-3-4-5 et un event le 4)
Doublon ^
J'ai tout réécrit en utilisant les locales built-in, ça devrait être plus propre en passant.
changed this line in version 17 of the diff
changed this line in version 17 of the diff
changed this line in version 17 of the diff
added 1 commit
1e3850bb
- Nettoyage dates et calendrierCompare with previous version
added 1 commit
128a9e32
- I18n des menusCompare with previous version
Il y avait aussi un problème avec l'internationalisation des menus (cf dernier commit), le seul moyen que j'ai trouvé pour le résoudre est un
.specific
dans le rendu des menus, c'est un peu bourrin :/...added 190 commits
master
712588af
- Merge branch 'master' into evarin/site-cofCompare with previous version
🤔
On a aussi besoin de
Orderable
ici ?Tu peux mettre cette explication dans un commentaire sur place stp ?
Btw, c'est exactement ce que fait
Paginator.get_page()
à partir de Django 2.0.https://docs.djangoproject.com/en/2.1/topics/pagination/#django.core.paginator.Paginator.get_page
Oui si le DirectoryPage ne fait pas un tri alphabétique
C'est à se demander pourquoi ils ont attendu django2 pour le faire ^^"
Oupsi
changed this line in version 20 of the diff
added 1 commit
f105225a
- Commentaires dans le codeCompare with previous version
Même si c'est déjà des
Page
? Ces objets sont bien ordonnés nan ?Ah oui, tiens. Je pensais qu'ils devaient être Orderable pour ça.
changed this line in version 21 of the diff
added 1 commit
376cc963
- Clean, lint and fix little thingsCompare with previous version
changed this line in version 22 of the diff
added 31 commits
master
39eaf4b1
- style -- black + isort84c88dfd
- Merge branch 'master' into aureplop/site-cofa6bf1fc1
- Merge branch 'aureplop/site-cof' into 'evarin/site-cof'Compare with previous version
@delobell up ? Il reste des choses à changer ou c'est bon ?
@champeno on merge dans master et on passe ça sur dev.cof ?
Ok soyons fous
resolved all discussions
banzai
merged
mentioned in commit
512868ee14
mentioned in merge request !344