thubrecht/poulpe
1
0
Fork 0
Code Issues 5 Pull requests 1 Projects Releases Packages Wiki Activity Actions

Models #8

Merged
narmanli merged 21 commits from models into master 2017-03-05 13:26:42 +01:00
Conversation 70 Commits 21 Files changed 35 +995 -16
narmanli commented 2016-10-13 11:48:17 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

models et tests de event terminés

models et tests de event terminés
narmanli commented 2016-10-13 16:21:10 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

  • d646f77d - heritage de titre pour str de Activity
Added 1 commit: * d646f77d - heritage de `titre` pour __str__ de Activity
narmanli commented 2016-10-14 13:14:38 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 5 commits:

  • dc8fc771 - premier jet models equipement
  • a32c66dc - Merge branch 'models' of git.eleves.ens.fr:cof-geek/GestionEvenementiel into models_equipement
  • 6021cc5a - middlexare fix
  • 3c93b0fa - models de equipment done
  • 8cfdc08e - Merge branch 'models_equipement' into 'models'
Added 5 commits: * dc8fc771 - premier jet models equipement * a32c66dc - Merge branch 'models' of git.eleves.ens.fr:cof-geek/GestionEvenementiel into models_equipement * 6021cc5a - middlexare fix * 3c93b0fa - models de equipment done * 8cfdc08e - Merge branch 'models_equipement' into 'models'
mpepin commented 2016-10-14 18:57:47 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Pas de concaténation de strings, il faut utiliser la syntaxe avec % à la place

Pas de concaténation de strings, il faut utiliser la syntaxe avec `%` à la place
mpepin commented 2016-10-14 18:59:16 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Pourquoi définir une fonction ici ?

Pourquoi définir une fonction ici ?
mpepin commented 2016-10-14 19:00:20 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

C'est à dire ?

C'est à dire ?
mpepin commented 2016-10-14 19:03:03 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Il faudrait peut-être un help_text pour indiquer que le champ attend un nombre hexadécimal

Il faudrait peut-être un `help_text` pour indiquer que le champ attend un nombre hexadécimal
narmanli commented 2016-10-17 19:38:04 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

pour le validateur mais je vais modifier la façon de le faire dans la prochaine version anyway

pour le validateur mais je vais modifier la façon de le faire dans la prochaine version anyway
narmanli commented 2016-10-17 19:39:13 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Qu'il faut expliquer ce que ça veut dire mais je ne me souviens plus trop bien ce qu'on avait décidé

Qu'il faut expliquer ce que ça veut dire mais je ne me souviens plus trop bien ce qu'on avait décidé
mpepin commented 2016-10-17 19:43:20 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Mon commentaire n'était pas clair : je ne vois pas l'intérêt de définir la fonction is_hex_color pour un seul appel. J'écrirais

is_hex_color = re_search(..., value)
if not is_hex_color:
    ...
Mon commentaire n'était pas clair : je ne vois pas l'intérêt de définir la fonction `is_hex_color` pour un seul appel. J'écrirais is_hex_color = re_search(..., value) if not is_hex_color: ...
mpepin commented 2016-10-17 19:44:38 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ça sert à faire une distinction dans l'affichage : on ne met pas le même CSS sur les tags publics et les tags visibles seulement par les orgas

Ça sert à faire une distinction dans l'affichage : on ne met pas le même CSS sur les tags publics et les tags visibles seulement par les orgas
narmanli commented 2016-10-17 19:47:38 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Pas faux. Je me sert de django.core.validators.RegexValidator dans ma nouvelle version

Pas faux. Je me sert de django.core.validators.RegexValidator dans ma nouvelle version
mpepin commented 2016-10-17 19:51:02 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ça règle le problème

Ça règle le problème
narmanli commented 2016-10-17 20:00:21 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 8 commits:

  • 8cfdc08e...855b3533 - 3 commits from branch master
  • 2a8b3ed3 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/GestionEvenementiel into models
  • cc9fea5b - Merge branch 'master' of git.eleves.ens.fr:cof-geek/GestionEvenementiel into models
  • f06ae696 - validator modified
  • 895ae3df - Merge branch 'models' of git.eleves.ens.fr:cof-geek/GestionEvenementiel into models
  • f3180c0b - small fixes
Added 8 commits: * 8cfdc08e...855b3533 - 3 commits from branch `master` * 2a8b3ed3 - Merge branch 'master' of git.eleves.ens.fr:cof-geek/GestionEvenementiel into models * cc9fea5b - Merge branch 'master' of git.eleves.ens.fr:cof-geek/GestionEvenementiel into models * f06ae696 - validator modified * 895ae3df - Merge branch 'models' of git.eleves.ens.fr:cof-geek/GestionEvenementiel into models * f3180c0b - small fixes
mpepin commented 2016-10-19 21:30:23 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

emprunté

emprunté
mpepin commented 2016-10-19 21:32:15 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Idem

Idem
mpepin commented 2016-10-19 21:34:55 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Il n'y a pas une typo au niveau du \ ?

->

"... des chiffres ou "
"les caractères ..."
Il n'y a pas une typo au niveau du `\` ? -> "... des chiffres ou " "les caractères ..."
mpepin commented 2016-10-19 21:35:58 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Enfin il faut supprimer la fonction si elle ne sert plus

Enfin il faut supprimer la fonction si elle ne sert plus
mpepin commented 2016-10-19 21:42:20 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

On ne peut pas dire que cette fonction soulève un exception quand on lui demande un attribut pas héritable ?

Je pense que ça aidera, au débogage notamment.

On ne peut pas dire que cette fonction soulève un exception quand on lui demande un attribut pas héritable ? Je pense que ça aidera, au débogage notamment.
mpepin commented 2016-10-19 21:42:43 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je pense qu'on pourrait virer ce morceau de code du coup.

Je pense qu'on pourrait virer ce morceau de code du coup.
mpepin commented 2016-10-19 21:44:43 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Je suppose que tu n'as pas fini

Je suppose que tu n'as pas fini
narmanli commented 2016-10-21 17:34:52 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Indeed, my bad

Indeed, my bad
narmanli commented 2016-10-21 17:35:40 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Si si, c'était juste pour que je sache quels champs il fallait que je teste. Je le vire.

Si si, c'était juste pour que je sache quels champs il fallait que je teste. Je le vire.
narmanli commented 2016-10-21 17:49:22 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Comme tu veux. D'un autre côté ça nous empêche de faire une boucle sur les fields. Tu préfères quoi ?

Comme tu veux. D'un autre côté ça nous empêche de faire une boucle sur les fields. Tu préfères quoi ?
narmanli commented 2016-10-21 18:30:15 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Enfin par "nous empêche" c'est juste que ça devient tout de suite plus relou de faire une telle boucle ^^

Enfin par "nous empêche" c'est juste que ça devient tout de suite plus relou de faire une telle boucle ^^
narmanli commented 2016-10-21 18:50:24 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * 926bb664 - small fixes
mpepin commented 2016-10-22 00:59:48 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Mouais enfin ça ne doit pas arriver tous les jours de boucler sur les attributs... En fait j'ai du mal à imaginer une bonne raison de le faire

Mouais enfin ça ne doit pas arriver tous les jours de boucler sur les attributs... En fait j'ai du mal à imaginer une bonne raison de le faire
narmanli commented 2017-02-01 12:46:43 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Resolved all discussions

Resolved all discussions
mpepin commented 2017-02-01 12:52:49 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Ou mieux : .format

Ou mieux : `.format`
narmanli commented 2017-02-01 13:03:11 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

J'ai utilisé la syntaxe avec %, cf le dernier commit d'il y a trois mois ^^

J'ai utilisé la syntaxe avec `%`, cf le dernier commit d'il y a trois mois ^^
mpepin commented 2017-02-01 13:21:48 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

C'était pas marqué résolu aussi :P

C'était pas marqué résolu aussi :P
narmanli commented 2017-02-01 13:24:52 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

" 10/10 discussions resolved " ;) du coup si tu veux faire de la review :P

" 10/10 discussions resolved " ;) du coup si tu veux faire de la review :P
narmanli commented 2017-02-18 01:44:37 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * 87967dba - adding user model
narmanli commented 2017-02-18 20:31:49 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in merge request !5

mentioned in merge request !5
mpepin commented 2017-02-21 16:27:17 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

if attrname in {"parent", "staff", "equipment"}: serait moins lourd

`if attrname in {"parent", "staff", "equipment"}:` serait moins lourd
mpepin commented 2017-02-21 16:32:26 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

pourquoi pas self.loge ici ?

pourquoi pas `self.loge` ici ?
mpepin commented 2017-02-21 16:32:34 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

idem

idem
mpepin commented 2017-02-21 16:32:42 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

idem

idem
mpepin commented 2017-02-21 16:32:46 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

idem

idem
mpepin commented 2017-02-21 16:35:02 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

typo : Clé

typo : Clé
mpepin commented 2017-02-21 16:35:34 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

typo : erronée

typo : erronée
mpepin commented 2017-02-21 16:46:12 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

C'est quoi l'intérêt d'utiliser ce formulaire si les champs {last,first}_name et email ne sont pas dedans ?

Quid de l'idée d'utiliser un ModelForm avec le champ key en plus ? Ça évite d'avoir à surcharger save pour faire du travail que Django pourrait faire tout seul.

C'est quoi l'intérêt d'utiliser ce formulaire si les champs `{last,first}_name` et `email` ne sont pas dedans ? Quid de l'idée d'utiliser un `ModelForm` avec le champ `key` en plus ? Ça évite d'avoir à surcharger `save` pour faire du travail que Django pourrait faire tout seul.
mpepin commented 2017-02-21 16:50:30 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Pourquoi ces commentaires ?

Pourquoi ces commentaires ?
mpepin commented 2017-02-21 16:51:59 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

je pense que 'erkan:index' est un copier-coller malheureux

je pense que `'erkan:index'` est un copier-coller malheureux
mpepin commented 2017-02-21 16:52:21 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Ce fichier n'est pas PEP8 compliant

Ce fichier n'est pas PEP8 compliant
mpepin commented 2017-02-21 16:53:05 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

C'est quoi ça ?

C'est quoi ça ?
mpepin commented 2017-02-21 16:55:08 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Est-il nécessaire de séparer autant les templates ? Typiquement là, je ne vois pas quoi d'autre pourrait hériter de "user/user_form.html" et je ne suis pas fan de l'idée d'ouvrir 5 fichiers différents pour changer un truc dans un template

Est-il nécessaire de séparer autant les templates ? Typiquement là, je ne vois pas quoi d'autre pourrait hériter de "user/user_form.html" et je ne suis pas fan de l'idée d'ouvrir 5 fichiers différents pour changer un truc dans un template
mpepin commented 2017-02-21 16:55:34 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Ah c'était au dessus…

Ah c'était au dessus…
mpepin commented 2017-02-21 16:56:27 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Je ne crois pas

Je ne crois pas
mpepin commented 2017-02-21 17:06:39 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Je ne suis pas très fan de ce formulaire très générique, j'ai l'impression que ça complique beaucoup les choses pour un gain assez faible.

Je ne suis pas très fan de ce formulaire très générique, j'ai l'impression que ça complique beaucoup les choses pour un gain assez faible.
mpepin commented 2017-02-21 17:08:04 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Il faudra aussi stash les migrations, l'idée de cette branche était de poser les bases d'un coup avec une migration par app

Il faudra aussi stash les migrations, l'idée de cette branche était de poser les bases d'un coup avec une migration par app
narmanli commented 2017-02-21 17:16:17 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * c7db0c10 - include forgotten
mpepin commented 2017-02-21 17:20:22 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Index n'existe pas

Index n'existe pas
mpepin commented 2017-02-21 17:21:23 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Il y a beaucoup d'erreurs PEP8 dans ce fichier

Il y a beaucoup d'erreurs PEP8 dans ce fichier
narmanli commented 2017-02-21 17:23:28 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Résolu dans un commit ultérieur

Résolu dans un commit ultérieur
mpepin commented 2017-02-21 17:34:23 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Il faudrait utiliser les chaînes pour regexp (e.g. r"...") .

  1. Pour l’homogénéité
  2. Pour ne pas avoir à échapper des symboles sans arrêt
  3. Pour éviter les mauvaises surprises
Il faudrait utiliser les chaînes pour regexp (e.g. `r"..."`) . 1. Pour l’homogénéité 2. Pour ne pas avoir à échapper des symboles sans arrêt 3. Pour éviter les mauvaises surprises
mpepin commented 2017-02-21 17:36:04 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Une erreur SQL fait que les tests ne peuvent pas être lancés.

Une erreur SQL fait que les tests ne peuvent pas être lancés.
narmanli commented 2017-02-21 19:00:34 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Il faudrait aussi gérer les widgets du coup c'est kiff-kiff

Il faudrait aussi gérer les widgets du coup c'est kiff-kiff
narmanli commented 2017-02-21 19:01:12 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Résolu dans un commit ultérieur

Résolu dans un commit ultérieur
narmanli commented 2017-02-21 19:01:19 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Résolu dans un commit ultérieur

Résolu dans un commit ultérieur
narmanli commented 2017-02-21 19:01:39 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

l'objet du mail de réinitialisation du mot de passe :)
Résolu dans un commit ultérieur

l'objet du mail de réinitialisation du mot de passe :) Résolu dans un commit ultérieur
narmanli commented 2017-02-21 19:01:51 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Résolu dans un commit ultérieur

Résolu dans un commit ultérieur
narmanli commented 2017-02-21 19:02:03 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Résolu dans un commit ultérieur

Résolu dans un commit ultérieur
narmanli commented 2017-02-21 19:06:11 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Résolu dans un commit ultérieur

Résolu dans un commit ultérieur
narmanli commented 2017-02-21 19:07:12 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Résolu dans un commit ultérieur

Résolu dans un commit ultérieur
narmanli commented 2017-02-21 20:43:22 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * 8055be4c - fixes
narmanli commented 2017-02-21 21:15:57 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * e72969e9 - initial migrations
mpepin commented 2017-02-24 20:21:08 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Il faut enlever le .name du coup

Il faut enlever le `.name` du coup
narmanli commented 2017-03-05 13:24:16 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Added 1 commit:

Added 1 commit: * 01014c68 - test fixed
mpepin commented 2017-03-05 13:26:31 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Status changed to merged

Status changed to merged
mpepin commented 2017-03-05 13:26:42 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in commit bbb5123051

mentioned in commit bbb51230519ff63bfbdcbd9f2b3e69ecb20be0ff
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Doing

   
Doing

   
Good first issue

   
To Do
No labels
Doing
Doing
Good first issue
To Do
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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: thubrecht/poulpe#8
No description provided.
Delete branch "models"

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?