Produits Entr'ouvert » Chrono

Pour préciser le côté pratique/UI de la chose, ça pourrait être étendre la popup de chargement des exceptions, commencer par une liste des sources d'exception existantes et ensuite avoir la définition d'une nouvelle source

Exceptions (titre)

* https://.../blah.ics               [rafraichir] [supprimer]
* FrenchHolidays.ics                  [remplacer] [supprimer]

Pour ajouter de nouvelles exceptions en masse, vous pouvez
télécharger un fichier ou spécifier l'adresse d'un calendrier
distant.

* Fichier ICS: [..............]
* URL: [......................]

                                       [Annuler]  [Importer]

Je trouve ça très bien.

Si tu supprimes toutes les exceptions à chaque chargement, tu peux complètement virer la partie keep_synced_by_uid elle
servait justement à gérer le rafraîchissement en l'absence d'objet « source », c'est donc devenu inutile.

Purement pour la forme: pour la vue de rafraîchissement c'est toujours plus joli et plus respectueux de la séparation vue/formulaire de mettre toute la validation dans Form.clean() plutôt que d'en déplacer des bouts dans form_valid() (ça oblige à des form.add_error(None, ...) un peu moche).

À propos de 0001, je me dis qu'on a décidé black pour chrono (#37464), que même sans l'infra pour vérifier côté jenkins que ça colle, on pourrait déjà faire la passe sur le code . (dans le commit de la branche de #37464, j'ai mis black --target-version py35 --skip-string-normalization --line-length 110, ça peut s'y discuter).

Je pense que je ferais sans la confirmation lors du "refresh".

(et j'allais dire que ma version de black ne produit pas le même résultat, mais en fait c'est que tu as limité le changement aux fichiers que tu touchais, je te dirais d'y aller totalement, que tous les fichiers soient traités une fois pour toutes).

Lauréline Guerin a écrit :

J'ai déplacé un check dans la méthode clean du form ExceptionsImportForm, mais j'ai laissé la gestion des ICSError dans les vues.

Je parlais de TimePeriodExceptionSourceRefreshForm en fait.

ExceptionsImportForm ou TimePeriodExceptionSourceRefreshForm, c'est la même mécanique: on valide le form, on save l'instance, puis on lance l'import en form_valid