management commands: share support for --all-tenants (#84868) #946

Merged
fpeters merged 1 commits from wip/84868-commands-all-tenants into main 2023-12-19 14:08:38 +01:00
Owner
No description provided.
fpeters force-pushed wip/84868-commands-all-tenants from 7bef123ff5 to babef6f7c8 2023-12-18 19:58:40 +01:00 Compare
fpeters force-pushed wip/84868-commands-all-tenants from babef6f7c8 to 88f273362a 2023-12-18 20:27:17 +01:00 Compare
fpeters changed title from WIP: management commands: share support for --all-tenants (#84868) to management commands: share support for --all-tenants (#84868) 2023-12-19 08:22:40 +01:00
pmarillonnet reviewed 2023-12-19 11:48:29 +01:00
@ -26,0 +39,4 @@
if domain and all_tenants:
raise CommandError('--domain and --all-tenants are exclusive')
if not (domain or all_tenants):
raise CommandError('--domain or --all-tenants are required')
Owner

J’en profiterais pour corriger l’erreur d’accord (is required, singulier ici, d’autant plus que les deux options sont mutuellement exclusives.)

J’en profiterais pour corriger l’erreur d’accord (`is required`, singulier ici, d’autant plus que les deux options sont mutuellement exclusives.)
Owner

Mais en fait c’est totalement du détail et ne doit pas être bloquant, le reste me convient. Remarque à prendre ou pas. Ack.

Mais en fait c’est totalement du détail et ne doit pas être bloquant, le reste me convient. Remarque à prendre ou pas. Ack.
Author
Owner

je corrige même en "either --domain or --all-tenants is required".

je corrige même en "either --domain or --all-tenants is required".
pmarillonnet approved these changes 2023-12-19 12:20:09 +01:00
fpeters force-pushed wip/84868-commands-all-tenants from 88f273362a to 482f5968ee 2023-12-19 13:53:24 +01:00 Compare
fpeters merged commit a82e06d843 into main 2023-12-19 14:08:38 +01:00
fpeters deleted branch wip/84868-commands-all-tenants 2023-12-19 14:08:38 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
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: entrouvert/wcs#946
No description provided.