Development #65442
Tarification - vue pour importer un site
0%
Description
On a les vues d'export qui étaient implémentées dans chrono pour la tarification, mais l'import dans chrono se faisait sur la page d'accueil agendas;
Faire un import/export sur la page d'accueil de la tarification ? (export = export de toute la tarification, import = comme l'import dans chrono)
Fichiers
Révisions associées
pricing: remove _as_manager tests (#65442)
pricing: simplify mixins and permission checks (#65442)
Historique
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Sujet changé de Tarification - pour importer un site à Tarification - vue pour importer un site
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Fichier 0003-pricing-simplify-mixins-and-permission-checks-65442.patch 0003-pricing-simplify-mixins-and-permission-checks-65442.patch ajouté
- Fichier 0002-pricing-remove-_as_manager-tests-65442.patch 0002-pricing-remove-_as_manager-tests-65442.patch ajouté
- Fichier 0001-pricing-import-export-65442.patch 0001-pricing-import-export-65442.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
0001: import/export global
J'ai rangé ça dans l'app pricing, même si ça parle un peu d'agendas, parce que ça concerne quand même la tarification.
De même, j'ai ajouté les boutons import/export sur /manage/pricing/ et non sur /manage/
0002: nettoyage des tests
0003: et simplification des mixins agendas, puisque pour le moment seul un admin peut accéder à tout ça.
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Fichier 0003-pricing-simplify-mixins-and-permission-checks-65442.patch 0003-pricing-simplify-mixins-and-permission-checks-65442.patch ajouté
- Fichier 0002-pricing-remove-_as_manager-tests-65442.patch 0002-pricing-remove-_as_manager-tests-65442.patch ajouté
- Fichier 0001-pricing-import-export-65442.patch 0001-pricing-import-export-65442.patch ajouté
Mis à jour par Lauréline Guérin il y a presque 2 ans
Mis à jour par Lauréline Guérin il y a presque 2 ans
Mis à jour par Thomas Noël il y a presque 2 ans
- sans doute qu'il manque un "pricing" dans le nom des vues déclarées, je vois "lingo-manager-config-export" et "lingo-manager-config-import" : peut-être remplacer "config" par "pricing" ou "pricing-config" ?
- sur les commandes de manage export_site/import_site, pareil je pense qu'on peut les renommer export-pricing/import-pricing, car dans lingo on aura d'autres choses que la tarification (régies, systèmes paiements, etc) à exporter et importer. Mais je crois qu'on peut aussi décider de se passer de ces commandes pour l'instant -- je n'en vois pas trop l'usage.
0002 : ok, vive le rouge !
0003 : je ne suis pas un grand fan des protection de vues ajoutées dans urls.py, je préférerais voir ça dans le views.py, mais bon, hey, ça roule.
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Fichier 0003-pricing-simplify-mixins-and-permission-checks-65442.patch 0003-pricing-simplify-mixins-and-permission-checks-65442.patch ajouté
- Fichier 0002-pricing-remove-_as_manager-tests-65442.patch 0002-pricing-remove-_as_manager-tests-65442.patch ajouté
- Fichier 0001-pricing-import-export-65442.patch 0001-pricing-import-export-65442.patch ajouté
- Assigné à mis à Lauréline Guérin
- ok, renommé en lingo-manager-pricing-config-import et lingo-manager-pricing-config-export
- ok, renommé en export_pricing_config et import_pricing_config; je les ai gardées parce qu'il y a une petite option qui permet de vider un site, comme sur chrono. Dans le doute, et pour bien tester ce comportement, j'ai laissé
0003: ça fait moins de code :) J'aurais pu écrire un mixin, mais alors il est facile d'oublier de l'utiliser (comme il est facile d'oublier de rajouter le dispatch qui check les permissions) (mais on pourrait ajouter un test qui boucle sur tout les urls du module et les teste une a une; flemme). Là on voit en lisant urls.py que toutes les urls sont protégées.
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Lauréline Guerin a écrit :
0001:
- ok, renommé en lingo-manager-pricing-config-import et lingo-manager-pricing-config-export
- ok, renommé en export_pricing_config et import_pricing_config; je les ai gardées parce qu'il y a une petite option qui permet de vider un site, comme sur chrono. Dans le doute, et pour bien tester ce comportement, j'ai laissé
Ca marche.
0003: ça fait moins de code :) J'aurais pu écrire un mixin, mais alors il est facile d'oublier de l'utiliser (comme il est facile d'oublier de rajouter le dispatch qui check les permissions) (mais on pourrait ajouter un test qui boucle sur tout les urls du module et les tests une a une; flemme). Là on voit en lisant urls.py que toutes les urls sont protégées.
C'est vrai. C'est mieux, en fait.
Je n'ai plus rien dire ; ack.
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 1ee53fa3bedab1e50e0b248b3f24eb8b7b3567d3
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri May 20 09:59:06 2022 +0200
pricing: simplify mixins and permission checks (#65442)
commit 6f093e0c1ad0504525624858d0d85758f040f43f
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri May 20 09:50:28 2022 +0200
pricing: remove _as_manager tests (#65442)
commit 373560728a40b3bd999a25040823ba379e6816e1
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Thu May 19 17:39:54 2022 +0200
pricing: import/export (#65442)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
pricing: import/export (#65442)