Gleipnir/nidus-sync
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix notification counts being off

Browse source 
Double-counting water
This commit is contained in:
Eli Ribble 2026-03-14 16:12:52 +00:00
parent 1a9a72adc0
commit 66e122f7e7
No known key found for this signature in database
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
platform/notification.go
View file

@ -111,7 +111,7 @@ func NotificationCountsForUser(ctx context.Context, u User) (*UserNotificationCo
if err != nil {
return nil, fmt.Errorf("Failed to get home notification count: %w", err)
}
count_nuisance, err := u.Organization.model.Waters().Count(ctx, db.PGInstance.BobDB)
count_nuisance, err := u.Organization.model.Nuisances().Count(ctx, db.PGInstance.BobDB)
if err != nil {
return nil, fmt.Errorf("Failed to get nuisance notification count: %w", err)
}
@ -119,6 +119,7 @@ func NotificationCountsForUser(ctx context.Context, u User) (*UserNotificationCo
if err != nil {
return nil, fmt.Errorf("Failed to get water notification count: %w", err)
}
//log.Debug().Int64("nuisance", count_nuisance).Int64("water", count_water).Int64("home", count_home).Int("user", u.ID).Msg("calculated notification counts")
return &UserNotificationCounts{
Communications: uint(count_nuisance + count_water),
Home: uint(count_home),