Skip to content

Commit

Permalink
Merge pull request #945 from gobitfly/BEDS-592/getMessageCount_wrong_…
Browse files Browse the repository at this point in the history
…assignment

fix(notifications): getMessageCount assignment is wrong
  • Loading branch information
peterbitfly authored Oct 11, 2024
2 parents eaede6a + 293db0b commit e5e12fd
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions backend/pkg/api/data_access/notifications.go
Original file line number Diff line number Diff line change
Expand Up @@ -186,23 +186,24 @@ func (d *DataAccessService) GetNotificationOverview(ctx context.Context, userId
var err error
day := time.Now().Truncate(utils.Day).Unix()
getMessageCount := func(prefix string) (uint64, error) {
res := d.persistentRedisDbClient.Get(ctx, fmt.Sprintf("%s:%d:%d", prefix, userId, day))
key := fmt.Sprintf("%s:%d:%d", prefix, userId, day)
res := d.persistentRedisDbClient.Get(ctx, key)
if res.Err() == redis.Nil {
return 0, nil
} else if res.Err() != nil {
return 0, res.Err()
}
return res.Uint64()
}
response.Last24hPushCount, err = getMessageCount("n_mails")
response.Last24hEmailsCount, err = getMessageCount("n_mails")
if err != nil {
return err
}
response.Last24hPushCount, err = getMessageCount("n_push")
if err != nil {
return err
}
response.Last24hPushCount, err = getMessageCount("n_webhook")
response.Last24hWebhookCount, err = getMessageCount("n_webhook")
return err
})

Expand Down

0 comments on commit e5e12fd

Please sign in to comment.