entrouvert
/
chrono
15
1
Fork 0
Code Issues Pull Requests 10 Releases Wiki Activity

manager: hide internal roles in agenda settings (#77155) #80

Merged
vdeniaud merged 1 commits from wip/77155-Les-roles-techniques-prefixes-av into main 2023-05-17 16:31:34 +02:00
Conversation 0 Commits 1 Files Changed 2
vdeniaud commented 2023-05-11 17:46:12 +02:00
Owner

Pas l'impression de pouvoir tester le code, puisque le modèle Role n'existe pas dans les tests.

Pas l'impression de pouvoir tester le code, puisque le modèle Role n'existe pas dans les tests.
vdeniaud added 1 commit 2023-05-11 17:46:12 +02:00
gitea/chrono/pipeline/head This commit looks good Details
0129bba6ad manager: hide internal roles in agenda settings (#77155)
ecazenave requested changes 2023-05-11 18:01:32 +02:00
chrono/manager/utils.py
@ -127,0 +130,4 @@
def get_role_queryset():
qs = Group.objects.all().order_by('name')
try:
ecazenave commented 2023-05-11 18:01:05 +02:00
Owner

Mais pourquoi tant de précautions ? (je ne vois pas comment ça pourrait foirer)

Mais pourquoi tant de précautions ? (je ne vois pas comment ça pourrait foirer)
vdeniaud commented 2023-05-11 18:10:21 +02:00
Poster
Owner

C'est noté en description de la PR, le modèle Role n'existe pas dans les tests (on a pas accès à la machinerie hobo). Donc ne pas faire ça -> plein de tests qui foirent sur « pas de champ role ».

C'est noté en description de la PR, le modèle Role n'existe pas dans les tests (on a pas accès à la machinerie hobo). Donc ne pas faire ça -> plein de tests qui foirent sur « pas de champ role ».
vdeniaud requested review from ecazenave 2023-05-11 18:10:26 +02:00
ecazenave approved these changes 2023-05-17 16:21:26 +02:00
vdeniaud merged commit f0041d5ad8 into main 2023-05-17 16:31:34 +02:00
vdeniaud deleted branch wip/77155-Les-roles-techniques-prefixes-av 2023-05-17 16:31:34 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
ecazenave
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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/chrono#80
There is no content yet.
Delete Branch "wip/77155-Les-roles-techniques-prefixes-av"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?