Kerl/fix dev data #464

Merged
mpepin merged 0 commits from Kerl/fix_dev_data into master 2017-02-07 11:00:59 +01:00
mpepin commented 2017-02-03 17:27:57 +01:00 (Migrated from git.eleves.ens.fr)

Les données de développement sont désormais réparties entre deux fixtures et un commande django-admin.

  • Fixtures
    • gestion : sondages, événéments, matières de petits cours et deux demandes de petits cours.
    • sites : le site pour l'application sites.
  • La commande loaddevdata : elle se découpe en deux parties.
    • La première dont le code est dans gestioncof/management/commands/loaddevdata.py charge les utilisateurs dans la base de données et les inscrits aux petits cours. Si l'argument --no-root est passé, l'utilisateur privilégié root n'est pas créé.
    • La deuxième, loadbdadevdata est appelée par la première. Elle instancie deux tirages et y inscrit tous les utilisateurs puis lance le premier tirage.

Comme on peut le constater dans provisioning/prepare_django.sh, les commandes à appeler pour setup l'environnement de dev sont

python manage.py loaddata gestion sites
python manage.py loaddevdata

Pour augmenter la liste des données à importer, compléter les fichiers contenus dans {gestioncof,bda}/management/data/

Les données de développement sont désormais réparties entre deux fixtures et un commande django-admin. - Fixtures - `gestion` : sondages, événéments, matières de petits cours et deux demandes de petits cours. - `sites` : le site pour l'application `sites`. - La commande `loaddevdata` : elle se découpe en deux parties. - La première dont le code est dans `gestioncof/management/commands/loaddevdata.py` charge les utilisateurs dans la base de données et les inscrits aux petits cours. Si l'argument `--no-root` est passé, l'utilisateur privilégié `root` n'est pas créé. - La deuxième, `loadbdadevdata` est appelée par la première. Elle instancie deux tirages et y inscrit tous les utilisateurs puis lance le premier tirage. Comme on peut le constater dans `provisioning/prepare_django.sh`, les commandes à appeler pour setup l'environnement de dev sont python manage.py loaddata gestion sites python manage.py loaddevdata Pour augmenter la liste des données à importer, compléter les fichiers contenus dans `{gestioncof,bda}/management/data/`
mpepin commented 2017-02-03 17:29:54 +01:00 (Migrated from git.eleves.ens.fr)

Added 1 commit:

Added 1 commit: * 45856ca8 - update README
lstephan commented 2017-02-03 22:09:08 +01:00 (Migrated from git.eleves.ens.fr)

2 remarques :

  • J'ai une erreur avec l'option --no-root, qui n'est pas reconnue (manage.py loaddevdata: error: unrecognized arguments: --no-root)
  • Pour la symétrie, je serais pour mettre loaddata dans cof/, et séparer la partie loadbdadevdata et loadgestiondevdata (et plus tard loadkfetdevdata ?)
2 remarques : - J'ai une erreur avec l'option `--no-root`, qui n'est pas reconnue (`manage.py loaddevdata: error: unrecognized arguments: --no-root`) - Pour la symétrie, je serais pour mettre `loaddata` dans `cof/`, et séparer la partie `loadbdadevdata` et `loadgestiondevdata` (et plus tard `loadkfetdevdata` ?)
mpepin commented 2017-02-03 23:41:52 +01:00 (Migrated from git.eleves.ens.fr)

Added 1 commit:

Added 1 commit: * 13da42b8 - typo
mpepin commented 2017-02-03 23:44:41 +01:00 (Migrated from git.eleves.ens.fr)
  • C'était une typo, normalement là ça marche
  • Très bon point mais apparemment les commandes django-admin doivent être dans des apps django (comme gestioncof ou bda) or cof n'est pas une app, juste un module avec des fichiers de config.
- C'était une typo, normalement là ça marche - Très bon point mais apparemment les commandes django-admin doivent être dans des apps django (comme `gestioncof` ou `bda`) or `cof` n'est pas une app, juste un module avec des fichiers de config.
lstephan commented 2017-02-04 00:02:16 +01:00 (Migrated from git.eleves.ens.fr)

C'est vrai que de toute façon gestioncof a un statut d'app "principale" (avec home.html, etc) donc ça me va. L'option marche maintenant, tout me paraît bon.

C'est vrai que de toute façon `gestioncof` a un statut d'app "principale" (avec `home.html`, etc) donc ça me va. L'option marche maintenant, tout me paraît bon.
mpepin commented 2017-02-07 11:00:41 +01:00 (Migrated from git.eleves.ens.fr)

Comptons ça comme de la review, thx @lstephan :)

Comptons ça comme de la review, thx @lstephan :)
mpepin commented 2017-02-07 11:00:57 +01:00 (Migrated from git.eleves.ens.fr)

mentioned in commit 217624fe08

mentioned in commit 217624fe08b108e6f11d3499a24b02aee0e234d9
mpepin commented 2017-02-07 11:00:59 +01:00 (Migrated from git.eleves.ens.fr)

Status changed to merged

Status changed to merged
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#464
No description provided.