api: add new filters to get overlapping subscriptions (#71579) #9
vdeniaud
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/71579-api-subscriptions-overlapping"
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?
@ -2254,2 +2254,4 @@date_start = filters.DateFilter(lookup_expr='gte')date_end = filters.DateFilter(lookup_expr='lt')date_start_overlaps = filters.DateFilter(method='do_nothing')date_end_overlaps = filters.DateFilter(method='do_nothing')Pas super convaincue par l'ajout de nouveaux filtres date_xx_overlaps.
Plutôt changer le comportement des filtres date_start et date_end existants ?
Si on n'a rien qui les utilise, oui pour changer le comportement.
OK, ce n'est pas utilisé dans publik-famille, on va dire que ce n'est pas utilisé.
3f5ca156a3to3a36f5567c@ -2265,0 +2273,4 @@missing = [k for k, v in overlaps.items() if not v][0]not_missing = [k for k, v in overlaps.items() if v][0]raise ValidationError({missing: _('This filter is required when using "%s" filter.') % not_missing}J'ai l'impression qu'il y a moyen de déléguer la validation de filtres dépendants l'un de l'autre à django-filters https://rpkilby.github.io/django-filter/guide/usage.html#process-filters-together-with-meta-groups.
Ca aurait été bien, mais je ne trouve pas les groups dans le code ni dans la doc officielle: https://django-filter.readthedocs.io/en/main/guide/usage.html#the-filter
Zut j'ai pas vérifié que c'était la doc officielle, c'est la faute à Google !