Nouveau site du COF #551

Merged
champeno merged 0 commits from evarin/site-cof into master 2019-01-14 21:23:12 +01:00
champeno commented 2017-08-29 18:11:36 +02:00 (Migrated from git.eleves.ens.fr)

Un tout nouveau site pour le COF

TODO restant :

  • Couleurs (ping Erkan)
  • Calendrier dynamique (-> faire une API pour les events ? ou juste une view JSON à la main)
  • Affichage des news ?
  • CSS responsif
  • Pagination
  • I18N et switch EN/FR
  • Mieux gérer Évènement / Actualités simples (notamment QuerySet)
  • Page et pseudo-captcha listes mails
Un tout nouveau site pour le COF TODO restant : * [x] Couleurs (ping Erkan) * [x] Calendrier dynamique (-> faire une API pour les events ? ou juste une view JSON à la main) * [x] Affichage des news ? * [x] CSS responsif * [x] Pagination * [x] I18N et switch EN/FR * [x] Mieux gérer Évènement / Actualités simples (notamment QuerySet) * [x] Page et pseudo-captcha listes mails
champeno commented 2017-09-27 23:59:25 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit * 0e19abb5 - Cleaner homepage [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=657&start_sha=38af50a866989673837b73ea7c777e93134a4464)
champeno commented 2017-10-10 11:25:48 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit * 5a22b1cd - Affichage des actus [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=664&start_sha=0e19abb51a77eb3eade2970928c0e725099affdb)
champeno commented 2017-10-10 11:29:32 +02:00 (Migrated from git.eleves.ens.fr)

unmarked as a Work In Progress

unmarked as a **Work In Progress**
champeno commented 2017-10-10 11:29:32 +02:00 (Migrated from git.eleves.ens.fr)

changed the description

changed the description
champeno commented 2017-10-10 13:24:12 +02:00 (Migrated from git.eleves.ens.fr)

changed the description

changed the description
champeno commented 2017-10-23 11:06:01 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit * adf43889 - Fixtures site cof [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=684&start_sha=5a22b1cd372c12d447db7610fa591143ead201d6)
champeno commented 2018-01-20 19:37:41 +01:00 (Migrated from git.eleves.ens.fr)

added 113 commits

  • adf43889...44eee9be - 111 commits from branch master
  • ea495e8f - Archives beta
  • 8551ffcf - Merge branch 'master' into evarin/site-cof

Compare with previous version

added 113 commits * adf43889...44eee9be - 111 commits from branch `master` * ea495e8f - Archives beta * 8551ffcf - Merge branch 'master' into evarin/site-cof [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=759&start_sha=adf43889e1975d9e33f4580d6f19a9c5968ebbf5)
champeno commented 2018-01-22 21:24:34 +01:00 (Migrated from git.eleves.ens.fr)

added 1 commit

  • 8488beeb - Un seul modèle pour les actus

Compare with previous version

added 1 commit * 8488beeb - Un seul modèle pour les actus [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=762&start_sha=8551ffcfd3d4f0082e4bbeaed6ad91c9ea7def7f)
champeno commented 2018-01-28 17:12:31 +01:00 (Migrated from git.eleves.ens.fr)

marked the checklist item Mieux gérer Évènement / Actualités simples (notamment QuerySet) as completed

marked the checklist item **Mieux gérer Évènement / Actualités simples (notamment QuerySet)** as completed
champeno commented 2018-01-28 19:10:43 +01:00 (Migrated from git.eleves.ens.fr)

added 2 commits

Compare with previous version

added 2 commits * c11ccf2e - Tri des annuaires * f8952225 - Apparence et Responsiveness [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=768&start_sha=8488beeb4ef3fefc274bb10735bd2bc3041f7172)
champeno commented 2018-01-28 23:46:04 +01:00 (Migrated from git.eleves.ens.fr)

added 2 commits

  • ba870446 - Calendrier dynamique + sympa + jolies dates
  • 88e911ff - Fixtures mises à jour

Compare with previous version

added 2 commits * ba870446 - Calendrier dynamique + sympa + jolies dates * 88e911ff - Fixtures mises à jour [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=769&start_sha=f8952225d69e70aa348c175b99453687ff10bcc7)
champeno commented 2018-02-03 22:35:38 +01:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit * 59116f2d - International wagtail urls [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=772&start_sha=88e911ff9d76ec386286f344827e9e1f57b724fb)
champeno commented 2018-02-03 22:35:48 +01:00 (Migrated from git.eleves.ens.fr)

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 **Calendrier dynamique (-> faire une API pour les events ? ou juste une view JSON à la main)** as completed
champeno commented 2018-02-03 22:35:50 +01:00 (Migrated from git.eleves.ens.fr)

marked the checklist item Affichage des news ? as completed

marked the checklist item **Affichage des news ?** as completed
champeno commented 2018-02-03 22:35:52 +01:00 (Migrated from git.eleves.ens.fr)

marked the checklist item CSS responsif as completed

marked the checklist item **CSS responsif** as completed
champeno commented 2018-02-03 22:35:53 +01:00 (Migrated from git.eleves.ens.fr)

marked the checklist item Pagination as completed

marked the checklist item **Pagination** as completed
champeno commented 2018-02-03 22:35:54 +01:00 (Migrated from git.eleves.ens.fr)

marked the checklist item I18N et switch EN/FR as completed

marked the checklist item **I18N et switch EN/FR** as completed
champeno commented 2018-02-03 22:35:56 +01:00 (Migrated from git.eleves.ens.fr)

marked the checklist item Page et pseudo-captcha listes mails as completed

marked the checklist item **Page et pseudo-captcha listes mails** as completed
champeno commented 2018-02-03 22:40:16 +01:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit * 63ce694b - Fix dependency [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=773&start_sha=59116f2d46c633ceb03596a760ce53fcd624c0ed)
champeno commented 2018-02-20 17:13:31 +01:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit * 6d6c9955 - Fixtures fonctionnelles [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=790&start_sha=63ce694b4d3a992ed4bfa2991672307462c7de0d)
champeno commented 2018-02-20 17:16:34 +01:00 (Migrated from git.eleves.ens.fr)

TODO : changer le fonctionnement de la page "listes mails" pour faire comme en K-Fêt un captchablock ?
(Actuellement erreur liée à l'i18n)

TODO : changer le fonctionnement de la page "listes mails" pour faire comme en K-Fêt un captchablock ? (Actuellement erreur liée à l'i18n)
champeno commented 2018-02-20 17:21:35 +01:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit * 34749760 - Revert urls prefixing [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=791&start_sha=6d6c995563a5b984aad546291c5681661dde585b)
champeno commented 2018-03-21 21:55:45 +01:00 (Migrated from git.eleves.ens.fr)

added 1 commit

  • 3d091f50 - Suppr captcha page, ajout block iframe

Compare with previous version

added 1 commit * 3d091f50 - Suppr captcha page, ajout block iframe [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=794&start_sha=347497602ccd83159f54e8ce78dfacdfbc722ac6)
champeno commented 2018-03-21 22:35:05 +01:00 (Migrated from git.eleves.ens.fr)

marked the checklist item Couleurs (ping Erkan) as completed

marked the checklist item **Couleurs (ping Erkan)** as completed
champeno commented 2018-03-21 22:35:59 +01:00 (Migrated from git.eleves.ens.fr)

Finalement, lien vers page élèves et block iframe pour les pages qui en ont besoin (aka demande petit cours) => tout est prêt

Finalement, lien vers page élèves et block iframe pour les pages qui en ont besoin (aka demande petit cours) => tout est prêt
mpepin commented 2018-04-24 12:38:33 +02:00 (Migrated from git.eleves.ens.fr)

Pas besoin du u en python3, les chaînes de caractères sont unicode par défaut

Pas besoin du `u` en python3, les chaînes de caractères sont unicode par défaut
mpepin commented 2018-04-24 12:41:08 +02:00 (Migrated from git.eleves.ens.fr)

On pourrait peut-être donner un nom plus informatif à ce fichier

On pourrait peut-être donner un nom plus informatif à ce fichier
mpepin commented 2018-04-24 12:41:11 +02:00 (Migrated from git.eleves.ens.fr)

On pourrait peut-être donner un nom plus informatif à ce fichier

On pourrait peut-être donner un nom plus informatif à ce fichier
mpepin commented 2018-04-24 12:41:31 +02:00 (Migrated from git.eleves.ens.fr)

Idem, voire même fusionner les 3 fichiers en fait

Idem, voire même fusionner les 3 fichiers en fait
mpepin commented 2018-04-24 12:42:03 +02:00 (Migrated from git.eleves.ens.fr)

Les parenthèses ne servent à rien

Les parenthèses ne servent à rien
mpepin commented 2018-04-24 12:45:18 +02:00 (Migrated from git.eleves.ens.fr)

en python3 on peut juste écrire super().get_context(request)

en python3 on peut juste écrire `super().get_context(request)`
mpepin commented 2018-04-24 12:53:06 +02:00 (Migrated from git.eleves.ens.fr)

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

Qu'est ce qui se passe si page est un entier invalide ? On pourrait utiliser les [validators](https://docs.djangoproject.com/en/1.11/ref/validators/) de django pour vérifier que page est bien un entier et prendre une valeur par défaut si ça fail
mpepin commented 2018-04-24 12:53:52 +02:00 (Migrated from git.eleves.ens.fr)

Ça éliminerait un try … except

Ça éliminerait un `try … except`
mpepin commented 2018-04-24 12:54:53 +02:00 (Migrated from git.eleves.ens.fr)

Par curiosité, ça veut dire quoi chapo ?

Par curiosité, ça veut dire quoi chapo ?
champeno commented 2018-04-24 17:02:08 +02:00 (Migrated from git.eleves.ens.fr)

Chapô (Journalisme) Texte court coiffant un article, généralement typographié en gras, pour amener le lecteur à entrer dans l’article.

Chapô (Journalisme) Texte court coiffant un article, généralement typographié en gras, pour amener le lecteur à entrer dans l’article.
champeno commented 2018-04-28 15:57:15 +02:00 (Migrated from git.eleves.ens.fr)

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ême try...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 )

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ême `try...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 )
champeno commented 2018-04-28 16:00:29 +02:00 (Migrated from git.eleves.ens.fr)

changed this line in version 14 of the diff

changed this line in [version 14 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=844&start_sha=3d091f50b58af2609e9e39341e17552744fb0bc6#585ba4aa1b0bf78d1b8c2f3582d4057c1366149b_174_174)
champeno commented 2018-04-28 16:00:34 +02:00 (Migrated from git.eleves.ens.fr)

changed this line in version 14 of the diff

changed this line in [version 14 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=844&start_sha=3d091f50b58af2609e9e39341e17552744fb0bc6#5909daf8cbde35689c19ce0ad4fff9decfccf7f7_1_0)
champeno commented 2018-04-28 16:00:34 +02:00 (Migrated from git.eleves.ens.fr)

changed this line in version 14 of the diff

changed this line in [version 14 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=844&start_sha=3d091f50b58af2609e9e39341e17552744fb0bc6#5909daf8cbde35689c19ce0ad4fff9decfccf7f7_1_0)
champeno commented 2018-04-28 16:00:34 +02:00 (Migrated from git.eleves.ens.fr)

changed this line in version 14 of the diff

changed this line in [version 14 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=844&start_sha=3d091f50b58af2609e9e39341e17552744fb0bc6#c87f2cb2f7b0d10ac168f31c747a557bc5492b7a_1_0)
champeno commented 2018-04-28 16:00:34 +02:00 (Migrated from git.eleves.ens.fr)

changed this line in version 14 of the diff

changed this line in [version 14 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=844&start_sha=3d091f50b58af2609e9e39341e17552744fb0bc6#9678d619ce4002b7a170bbc752d085ed73c53033_17_17)
champeno commented 2018-04-28 16:00:35 +02:00 (Migrated from git.eleves.ens.fr)

changed this line in version 14 of the diff

changed this line in [version 14 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=844&start_sha=3d091f50b58af2609e9e39341e17552744fb0bc6#9678d619ce4002b7a170bbc752d085ed73c53033_77_77)
champeno commented 2018-04-28 16:00:35 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit * 6d72644e - Clean code up [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=844&start_sha=3d091f50b58af2609e9e39341e17552744fb0bc6)
mpepin commented 2018-04-29 10:52:49 +02:00 (Migrated from git.eleves.ens.fr)

ok ça marche

ok ça marche
mpepin commented 2018-04-29 10:52:50 +02:00 (Migrated from git.eleves.ens.fr)

resolved all discussions

resolved all discussions
mpepin commented 2018-04-29 10:58:12 +02:00 (Migrated from git.eleves.ens.fr)

que signifie le debugged_ ici ?

que signifie le `debugged_` ici ?
mpepin commented 2018-04-29 11:57:22 +02:00 (Migrated from git.eleves.ens.fr)

Là on veut peut-être utiliser un format différent en fonction de la langue.

Là on veut peut-être utiliser un format différent en fonction de la langue.
mpepin commented 2018-04-29 11:58:23 +02:00 (Migrated from git.eleves.ens.fr)

Et je pense qu'on devrait aussi stocker ce format ("d F Y") dans cof.locale.fr et cof.locale.en

Et je pense qu'on devrait aussi stocker ce format (`"d F Y"`) dans `cof.locale.fr` et `cof.locale.en`
mpepin commented 2018-04-29 12:46:21 +02:00 (Migrated from git.eleves.ens.fr)

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 :

from django.db.models import When, Case, DateTimeField
from django.db.models.functions import Trunc

def trunc(field_name):
    return Trunc(field_name, "day", output_field=DateTimeField())

events = (
    COFActuPage.objects.live()
    .filter(date_start__range=[month_prestart, month_postend], is_event=True)
    .annotate(
        day=Case(
            When(date_end__isnull=False, then=trunc("date_end")),
            default=trunc("date_start"),
            output_field=DateTimeField(),
        )
    )
    .order_by("day")  # ou pas
)

Le curday dans ton code correspond au champ day ici

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 : ```python from django.db.models import When, Case, DateTimeField from django.db.models.functions import Trunc def trunc(field_name): return Trunc(field_name, "day", output_field=DateTimeField()) events = ( COFActuPage.objects.live() .filter(date_start__range=[month_prestart, month_postend], is_event=True) .annotate( day=Case( When(date_end__isnull=False, then=trunc("date_end")), default=trunc("date_start"), output_field=DateTimeField(), ) ) .order_by("day") # ou pas ) ``` Le `curday` dans ton code correspond au champ `day` ici
mpepin commented 2018-04-29 12:46:25 +02:00 (Migrated from git.eleves.ens.fr)

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 :

from django.db.models import When, Case, DateTimeField
from django.db.models.functions import Trunc

def trunc(field_name):
    return Trunc(field_name, "day", output_field=DateTimeField())

events = (
    COFActuPage.objects.live()
    .filter(date_start__range=[month_prestart, month_postend], is_event=True)
    .annotate(
        day=Case(
            When(date_end__isnull=False, then=trunc("date_end")),
            default=trunc("date_start"),
            output_field=DateTimeField(),
        )
    )
    .order_by("day")  # ou pas
)

Le curday dans ton code correspond au champ day ici

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 : ```python from django.db.models import When, Case, DateTimeField from django.db.models.functions import Trunc def trunc(field_name): return Trunc(field_name, "day", output_field=DateTimeField()) events = ( COFActuPage.objects.live() .filter(date_start__range=[month_prestart, month_postend], is_event=True) .annotate( day=Case( When(date_end__isnull=False, then=trunc("date_end")), default=trunc("date_start"), output_field=DateTimeField(), ) ) .order_by("day") # ou pas ) ``` Le `curday` dans ton code correspond au champ `day` ici
mpepin commented 2018-04-29 12:47:27 +02:00 (Migrated from git.eleves.ens.fr)

pourquoi pas simplement str(next_month.month) etc

pourquoi pas simplement `str(next_month.month)` etc
mpepin commented 2018-04-29 12:53:26 +02:00 (Migrated from git.eleves.ens.fr)

Mê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 ?

Mê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 ?
champeno commented 2018-06-30 15:38:23 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit * 6d6ba70b - CSS++ [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=877&start_sha=6d72644ee356fd000b519ef542a7ffbcc6d3bcdc)
champeno commented 2018-06-30 15:53:05 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit * 954a6fdb - Wagtail requirement update [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=878&start_sha=6d6ba70bd71eb29c99885fce14a394c8f0953f61)
delobell commented 2018-09-29 22:24:36 +02:00 (Migrated from git.eleves.ens.fr)

👋 @champeno :-)

Ça en est où ? T'as le temps de t'occuper des commentaires de Martin ?

:wave: @champeno :-) Ça en est où ? T'as le temps de t'occuper des commentaires de Martin ?
champeno commented 2018-09-29 23:03:07 +02:00 (Migrated from git.eleves.ens.fr)

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.

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.
delobell commented 2018-09-30 14:53:34 +02:00 (Migrated from git.eleves.ens.fr)

added ~62 ~60 ~26 labels

added ~62 ~60 ~26 labels
champeno commented 2018-10-14 13:20:35 +02:00 (Migrated from git.eleves.ens.fr)

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 de get_absolute_url). Il faudrait que je voie si ces problèmes ont été résolus dans les màj de wagtail et modeltranslation

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 de `get_absolute_url`). Il faudrait que je voie si ces problèmes ont été résolus dans les màj de wagtail et modeltranslation
champeno commented 2018-10-14 13:33:14 +02:00 (Migrated from git.eleves.ens.fr)

En fait il suffit de faire |date tout seul et il utilise les formats par défaut définis (ou pas) dans les settings.

En fait il suffit de faire `|date` tout seul et il utilise les formats par défaut définis (ou pas) dans les settings.
champeno commented 2018-10-14 14:28:31 +02:00 (Migrated from git.eleves.ens.fr)

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)

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)
champeno commented 2018-10-14 14:28:59 +02:00 (Migrated from git.eleves.ens.fr)

Doublon ^

Doublon ^
champeno commented 2018-10-14 15:49:11 +02:00 (Migrated from git.eleves.ens.fr)

J'ai tout réécrit en utilisant les locales built-in, ça devrait être plus propre en passant.

J'ai tout réécrit en utilisant les locales built-in, ça devrait être plus propre en passant.
champeno commented 2018-10-14 15:51:12 +02:00 (Migrated from git.eleves.ens.fr)

changed this line in version 17 of the diff

changed this line in [version 17 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=1000&start_sha=954a6fdb536046fc18c9c0d185f4fc3d2cc2043b#eb93bc5a8361e1fc6f840af1cd1fa6fe2e487510_17_17)
champeno commented 2018-10-14 15:51:13 +02:00 (Migrated from git.eleves.ens.fr)

changed this line in version 17 of the diff

changed this line in [version 17 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=1000&start_sha=954a6fdb536046fc18c9c0d185f4fc3d2cc2043b#e7f3afb59e9c14f8de05fcd20a2a7e094638fdf7_79_76)
champeno commented 2018-10-14 15:51:13 +02:00 (Migrated from git.eleves.ens.fr)

changed this line in version 17 of the diff

changed this line in [version 17 of the diff](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=1000&start_sha=954a6fdb536046fc18c9c0d185f4fc3d2cc2043b#e7f3afb59e9c14f8de05fcd20a2a7e094638fdf7_115_118)
champeno commented 2018-10-14 15:51:14 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

  • 1e3850bb - Nettoyage dates et calendrier

Compare with previous version

added 1 commit <ul><li>1e3850bb - Nettoyage dates et calendrier</li></ul> [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=1000&start_sha=954a6fdb536046fc18c9c0d185f4fc3d2cc2043b)
champeno commented 2018-10-14 16:29:42 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit <ul><li>128a9e32 - I18n des menus</li></ul> [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/247/diffs?diff_id=1001&start_sha=1e3850bb6be655564a55cba9fd30d123b2fa61b0)
champeno commented 2018-10-14 16:30:29 +02:00 (Migrated from git.eleves.ens.fr)

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 :/...

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 :/...
mpepin commented 2018-11-19 23:30:41 +01:00 (Migrated from git.eleves.ens.fr)

added 190 commits

  • 128a9e32...6524f89e - 189 commits from branch master
  • 712588af - Merge branch 'master' into evarin/site-cof

Compare with previous version

added 190 commits <ul><li>128a9e32...6524f89e - 189 commits from branch <code>master</code></li><li>712588af - Merge branch &#39;master&#39; into evarin/site-cof</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/247/diffs?diff_id=1037&start_sha=128a9e32c06251e09734075ef812d93956df0b62)
delobell commented 2019-01-05 14:17:45 +01:00 (Migrated from git.eleves.ens.fr)

🤔

:thinking:
delobell commented 2019-01-05 14:28:51 +01:00 (Migrated from git.eleves.ens.fr)

On a aussi besoin de Orderable ici ?

On a aussi besoin de `Orderable` ici ?
delobell commented 2019-01-05 14:31:06 +01:00 (Migrated from git.eleves.ens.fr)

Tu peux mettre cette explication dans un commentaire sur place stp ?

Tu peux mettre cette explication dans un commentaire sur place stp ?
delobell commented 2019-01-05 14:35:08 +01:00 (Migrated from git.eleves.ens.fr)

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

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
champeno commented 2019-01-05 18:29:13 +01:00 (Migrated from git.eleves.ens.fr)

Oui si le DirectoryPage ne fait pas un tri alphabétique

Oui si le DirectoryPage ne fait pas un tri alphabétique
champeno commented 2019-01-05 18:47:34 +01:00 (Migrated from git.eleves.ens.fr)

C'est à se demander pourquoi ils ont attendu django2 pour le faire ^^"

C'est à se demander pourquoi ils ont attendu django2 pour le faire ^^"
champeno commented 2019-01-05 18:48:23 +01:00 (Migrated from git.eleves.ens.fr)

Oupsi

Oupsi
champeno commented 2019-01-05 18:51:20 +01:00 (Migrated from git.eleves.ens.fr)

changed this line in version 20 of the diff

changed this line in [version 20 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/247/diffs?diff_id=1055&start_sha=712588af7dae56ae98d5edda6c50cea3b1593c2f#8e895942d8822799569e9beaf46800c6d9142fa0_4_4)
champeno commented 2019-01-05 18:51:20 +01:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit <ul><li>f105225a - Commentaires dans le code</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/247/diffs?diff_id=1055&start_sha=712588af7dae56ae98d5edda6c50cea3b1593c2f)
delobell commented 2019-01-05 19:27:13 +01:00 (Migrated from git.eleves.ens.fr)

Même si c'est déjà des Page ? Ces objets sont bien ordonnés nan ?

Même si c'est déjà des `Page` ? Ces objets sont bien ordonnés nan ?
champeno commented 2019-01-06 00:18:12 +01:00 (Migrated from git.eleves.ens.fr)

Ah oui, tiens. Je pensais qu'ils devaient être Orderable pour ça.

Ah oui, tiens. Je pensais qu'ils devaient être Orderable pour ça.
champeno commented 2019-01-06 00:18:13 +01:00 (Migrated from git.eleves.ens.fr)

changed this line in version 21 of the diff

changed this line in [version 21 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/247/diffs?diff_id=1057&start_sha=f105225abf0803390c8a640c4f221a6aef608c72#9678d619ce4002b7a170bbc752d085ed73c53033_150_152)
champeno commented 2019-01-06 00:18:13 +01:00 (Migrated from git.eleves.ens.fr)

added 1 commit

  • 376cc963 - Clean, lint and fix little things

Compare with previous version

added 1 commit <ul><li>376cc963 - Clean, lint and fix little things</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/247/diffs?diff_id=1057&start_sha=f105225abf0803390c8a640c4f221a6aef608c72)
champeno commented 2019-01-06 12:02:44 +01:00 (Migrated from git.eleves.ens.fr)

changed this line in version 22 of the diff

changed this line in [version 22 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/247/diffs?diff_id=1063&start_sha=376cc963436a53dfc1abad5f215f6896708a1f09#9678d619ce4002b7a170bbc752d085ed73c53033_80_80)
champeno commented 2019-01-06 12:02:45 +01:00 (Migrated from git.eleves.ens.fr)

added 31 commits

  • 376cc963...e0e75b53 - 28 commits from branch master
  • 39eaf4b1 - style -- black + isort
  • 84c88dfd - Merge branch 'master' into aureplop/site-cof
  • a6bf1fc1 - Merge branch 'aureplop/site-cof' into 'evarin/site-cof'

Compare with previous version

added 31 commits <ul><li>376cc963...e0e75b53 - 28 commits from branch <code>master</code></li><li>39eaf4b1 - style -- black + isort</li><li>84c88dfd - Merge branch &#39;master&#39; into aureplop/site-cof</li><li>a6bf1fc1 - Merge branch &#39;aureplop/site-cof&#39; into &#39;evarin/site-cof&#39;</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/247/diffs?diff_id=1063&start_sha=376cc963436a53dfc1abad5f215f6896708a1f09)
champeno commented 2019-01-14 20:32:26 +01:00 (Migrated from git.eleves.ens.fr)

@delobell up ? Il reste des choses à changer ou c'est bon ?

@delobell up ? Il reste des choses à changer ou c'est bon ?
delobell commented 2019-01-14 21:08:00 +01:00 (Migrated from git.eleves.ens.fr)

@champeno on merge dans master et on passe ça sur dev.cof ?

@champeno on merge dans master et on passe ça sur dev.cof ?
champeno commented 2019-01-14 21:10:55 +01:00 (Migrated from git.eleves.ens.fr)

Ok soyons fous

Ok soyons fous
mpepin commented 2019-01-14 21:21:19 +01:00 (Migrated from git.eleves.ens.fr)

resolved all discussions

resolved all discussions
mpepin commented 2019-01-14 21:22:54 +01:00 (Migrated from git.eleves.ens.fr)

banzai

banzai
mpepin commented 2019-01-14 21:23:12 +01:00 (Migrated from git.eleves.ens.fr)

merged

merged
mpepin commented 2019-01-14 21:23:13 +01:00 (Migrated from git.eleves.ens.fr)

mentioned in commit 512868ee14

mentioned in commit 512868ee14b373c44f8dca5d75efc08630784989
delobell commented 2019-01-14 22:56:16 +01:00 (Migrated from git.eleves.ens.fr)

mentioned in merge request !344

mentioned in merge request !344
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/gestioCOF#551
No description provided.