entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 35 Releases Activity

api/statistics: add inactivity-related events (#85790) #235

Merged
pmarillonnet merged 1 commits from wip/85790-api-statisticts-inactivity into main 2024-01-30 16:19:29 +01:00
Conversation 1 Commits 1 Files Changed 2 +169 -4
pmarillonnet commented 2024-01-23 11:09:31 +01:00
Owner
Copy Link
No description provided.
pmarillonnet force-pushed wip/85790-api-statisticts-inactivity from c6695bcc54 to 2ebe7b651a 2024-01-23 11:23:07 +01:00 Compare
pmarillonnet force-pushed wip/85790-api-statisticts-inactivity from 2ebe7b651a to 3c1e74c868 2024-01-23 11:36:26 +01:00 Compare
pmarillonnet changed title from WIP: api/statistics: add inactivity-related events (#85790) to api/statistics: add inactivity-related events (#85790) 2024-01-23 11:38:08 +01:00
vdeniaud requested changes 2024-01-23 16:27:51 +01:00
src/authentic2/api_views.py Outdated
@ -1763,0 +1771,4 @@
@stat(name=_('Inactivity alert count'))
def inactivity_alert(self, request):
return self.get_statistics(request, UserNotificationActivity)
vdeniaud commented 2024-01-23 16:27:29 +01:00
Owner
Copy Link

Ce return et son homologue plus haut ne sont pas couverts, pour moi il faut ajouter des tests qui correspondent à ces stats

Ce return et son homologue plus haut ne sont pas couverts, pour moi il faut ajouter des tests qui correspondent à ces stats
pmarillonnet commented 2024-01-23 16:31:32 +01:00
Author
Owner
Copy Link

Ce return et son homologue plus haut ne sont pas couverts, pour moi il faut ajouter des tests qui correspondent à ces stats

Ok j’estimais qu’un get_statistics sur un événement simple sans méthode custom de génération des statistiques pouvait passer comme ça, je pensais qu’il en était de même des autres événements mais le rapport de couverture Jenkins me donne tort, je vais revoir cela.

> Ce return et son homologue plus haut ne sont pas couverts, pour moi il faut ajouter des tests qui correspondent à ces stats Ok j’estimais qu’un get_statistics sur un événement simple sans méthode custom de génération des statistiques pouvait passer comme ça, je pensais qu’il en était de même des autres événements mais le rapport de couverture Jenkins me donne tort, je vais revoir cela.
pmarillonnet commented 2024-01-30 10:28:35 +01:00
Author
Owner
Copy Link

Ce return et son homologue plus haut ne sont pas couverts, pour moi il faut ajouter des tests qui correspondent à ces stats

Ok j’estimais qu’un get_statistics sur un événement simple sans méthode custom de génération des statistiques pouvait passer comme ça, je pensais qu’il en était de même des autres événements mais le rapport de couverture Jenkins me donne tort, je vais revoir cela.

Voilà qui devrait être mieux.

> > Ce return et son homologue plus haut ne sont pas couverts, pour moi il faut ajouter des tests qui correspondent à ces stats > > Ok j’estimais qu’un get_statistics sur un événement simple sans méthode custom de génération des statistiques pouvait passer comme ça, je pensais qu’il en était de même des autres événements mais le rapport de couverture Jenkins me donne tort, je vais revoir cela. Voilà qui devrait être mieux.
pmarillonnet changed title from api/statistics: add inactivity-related events (#85790) to WIP: api/statistics: add inactivity-related events (#85790) 2024-01-23 16:31:41 +01:00
pmarillonnet force-pushed wip/85790-api-statisticts-inactivity from 3c1e74c868 to b0ff303e5b 2024-01-30 10:23:47 +01:00 Compare
pmarillonnet changed title from WIP: api/statistics: add inactivity-related events (#85790) to api/statistics: add inactivity-related events (#85790) 2024-01-30 10:28:41 +01:00
pmarillonnet requested review from vdeniaud 2024-01-30 10:28:43 +01:00
vdeniaud approved these changes 2024-01-30 15:55:04 +01:00
vdeniaud left a comment
Owner
Copy Link

Top

Top
pmarillonnet force-pushed wip/85790-api-statisticts-inactivity from b0ff303e5b to b41cca7ec5 2024-01-30 16:15:05 +01:00 Compare
pmarillonnet commented 2024-01-30 16:15:51 +01:00
Author
Owner
Copy Link

Petit rebase manuel parce qu’il y a eu d7bfac4ed5 entre temps, je merge dès que Jenkins est d’accord avec tout cela.

Petit rebase manuel parce qu’il y a eu d7bfac4ed55779d5b17443b9219dfe29c0c36474 entre temps, je merge dès que Jenkins est d’accord avec tout cela.
pmarillonnet merged commit b41cca7ec5 into main 2024-01-30 16:19:29 +01:00
pmarillonnet deleted branch wip/85790-api-statisticts-inactivity 2024-01-30 16:19:29 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
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/authentic#235
No description provided.
Delete Branch "wip/85790-api-statisticts-inactivity"

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?