Améliorations site du COF #719

Merged
champeno merged 0 commits from Evarin/sitecof-improvements into master 2020-05-08 16:13:20 +02:00
champeno commented 2020-03-29 16:19:05 +02:00 (Migrated from git.eleves.ens.fr)

Nombreuses petites améliorations du site du COF :

  • Limite à 4 news sur la page d'accueil
  • Amélioration de l'aside block : scroll si besoin, listes mieux séparées
  • Viewport (oups)
  • Traduction
Nombreuses petites améliorations du site du COF : * Limite à 4 news sur la page d'accueil * Amélioration de l'aside block : scroll si besoin, listes mieux séparées * Viewport (oups) * Traduction
champeno commented 2020-03-29 16:19:31 +02:00 (Migrated from git.eleves.ens.fr)

changed the description

changed the description
lstephan commented 2020-04-22 20:59:03 +02:00 (Migrated from git.eleves.ens.fr)

Premières remarques :

  • je n'arrive pas à loader la fixture examplesite0.json, à cause d'une clef use_specific non reconnue

  • si je vire cette clef, le site charge mais :

    • je n'arrive à accéder à rien : j'ai l'impression que l'url est à /site mais je prends des erreurs COFPage matching query does not exist.

    • essayer de modifier les pages via l'interface Wagtail m'amène à l'url /cms/pages/9/, et à une erreur 500 cryptique : KeyError: 27

J'ai merdé quelque chose ?

Premières remarques : - je n'arrive pas à loader la fixture `examplesite0.json`, à cause d'une clef `use_specific` non reconnue - si je vire cette clef, le site charge mais : * je n'arrive à accéder à rien : j'ai l'impression que l'url est à `/site` mais je prends des erreurs `COFPage matching query does not exist`. * essayer de modifier les pages via l'interface Wagtail m'amène à l'url `/cms/pages/9/`, et à une erreur 500 cryptique : `KeyError: 27` J'ai merdé quelque chose ?
mpepin commented 2020-05-08 15:51:43 +02:00 (Migrated from git.eleves.ens.fr)

J'ai (péniblement) réussi à importer les pages qui sont en prod. Je ne fais pas de fixture parce que je pense que j'ai cassé beaucoup de choses dans ma bdd locale (permissions, groupes, propriétaires des pages, etc) dans le processus.

Mais je peux constater :

  • que la limite à 4 news fonctionne
  • que la vue mobile (viewport) est potable
  • que la traduction semble fonctionner

Je ne comprends pas le dernier point en revanche : "Amélioration de l'aside block : scroll si besoin, listes mieux séparées"

J'ai (péniblement) réussi à importer les pages qui sont en prod. Je ne fais pas de fixture parce que je pense que j'ai cassé beaucoup de choses dans ma bdd locale (permissions, groupes, propriétaires des pages, etc) dans le processus. Mais je peux constater : - que la limite à 4 news fonctionne - que la vue mobile (viewport) est potable - que la traduction semble fonctionner Je ne comprends pas le dernier point en revanche : "Amélioration de l'aside block : scroll si besoin, listes mieux séparées"
mpepin commented 2020-05-08 15:52:54 +02:00 (Migrated from git.eleves.ens.fr)

added 1 commit

Compare with previous version

added 1 commit <ul><li>1ada8645 - Black</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/merge_requests/415/diffs?diff_id=1594&start_sha=9dabab51dbd9040d2009707a9d3901a6d7aa105b)
mpepin commented 2020-05-08 16:03:12 +02:00 (Migrated from git.eleves.ens.fr)

Ah si le "aside block" c'est le machin à gauche sur la page des clubs right ? Il y avait effectivement un problème quand on a beaucoup de clubs et il est réglé.

Ah si le "aside block" c'est le machin à gauche sur la page des clubs right ? Il y avait effectivement un problème quand on a beaucoup de clubs et il est réglé.
mpepin commented 2020-05-08 16:04:40 +02:00 (Migrated from git.eleves.ens.fr)

added 52 commits

  • 1ada8645...d4a9e96e - 50 commits from branch master
  • 67d7dafc - Merge branch 'master' into Evarin/sitecof-improvements
  • 6384cfc7 - Update changelog

Compare with previous version

added 52 commits <ul><li>1ada8645...d4a9e96e - 50 commits from branch <code>master</code></li><li>67d7dafc - Merge branch &#39;master&#39; into Evarin/sitecof-improvements</li><li>6384cfc7 - Update changelog</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/merge_requests/415/diffs?diff_id=1595&start_sha=1ada8645b83bd719ad38effa0e67647623ff4535)
mpepin commented 2020-05-08 16:04:55 +02:00 (Migrated from git.eleves.ens.fr)

enabled an automatic merge when the pipeline for 6384cfc701 succeeds

enabled an automatic merge when the pipeline for 6384cfc7017a44b76dfeffe99c14fc2a0e291240 succeeds
champeno commented 2020-05-08 16:09:54 +02:00 (Migrated from git.eleves.ens.fr)

Yup c'est ça.

Désolé j'ai eu une grosse flemme de m'occuper des fixtures >< elles ont tendances à ne jamais marcher (j'avais beaucoup galéré à en faire à la base, et elles s'étaient très vites retrouvées incompatibles), je crois que c'est plus rapide de recréer les pages que de les importer.

Yup c'est ça. Désolé j'ai eu une grosse flemme de m'occuper des fixtures >< elles ont tendances à ne jamais marcher (j'avais beaucoup galéré à en faire à la base, et elles s'étaient très vites retrouvées incompatibles), je crois que c'est plus rapide de recréer les pages que de les importer.
champeno commented 2020-05-08 16:10:55 +02:00 (Migrated from git.eleves.ens.fr)

Aussi, il reste un truc à faire : le captcha des listes, puisque le cof a l'air de tenir à mettre un lien vers les ml.

Aussi, il reste un truc à faire : le captcha des listes, puisque le cof a l'air de tenir à mettre un lien vers les ml.
mpepin commented 2020-05-08 16:11:36 +02:00 (Migrated from git.eleves.ens.fr)

ouais c'est le problème des fixtures… Peut-être qu'on en refera quand le site bougera un peu moins ?

ouais c'est le problème des fixtures… Peut-être qu'on en refera quand le site bougera un peu moins ?
mpepin commented 2020-05-08 16:13:20 +02:00 (Migrated from git.eleves.ens.fr)

merged

merged
mpepin commented 2020-05-08 16:13:20 +02:00 (Migrated from git.eleves.ens.fr)

mentioned in commit 6e9dc03bc7

mentioned in commit 6e9dc03bc79bba013f08c24f306a8ee275131248
mpepin commented 2020-05-08 16:13:53 +02:00 (Migrated from git.eleves.ens.fr)

Je viens de merge la MR mais si tu fais une nouvelle MR avec le captcha je peux la regarder dans la foulée

Je viens de merge la MR mais si tu fais une nouvelle MR avec le captcha je peux la regarder dans la foulée
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#719
No description provided.