jammella/gancio.notification
3
0
Fork 0
Code Issues 7 Pull requests Projects 1 Releases 1 Packages Wiki Activity Actions

Fix events notifier SQL query #89

Merged
versus merged 6 commits from fix/event_notify_query into main 2026-01-21 10:30:42 +00:00
Conversation 1 Commits 6 Files changed 5 +380 -17
versus commented 2026-01-19 18:04:00 +00:00
Owner
Copy link
No description provided.
n reviewed 2026-01-20 09:39:06 +00:00
src/core/notifier/dailyEventNotifier.go Outdated
@ -75,3 +75,3 @@
notifiersWG.Wait()
if err := notifier.repository.UpdateEvents(events); err != nil {
if err := notifier.repository.UpdateEvents(events, time.Now()); err != nil {
n commented 2026-01-20 09:39:06 +00:00
Owner
Copy link

why pass time.Now(), compute inside

why pass time.Now(), compute inside
n marked this conversation as resolved
versus merged commit 0f2e02856f into main 2026-01-21 10:30:42 +00:00
versus deleted branch fix/event_notify_query 2026-01-21 10:30:43 +00:00
versus referenced this pull request from a commit 2026-01-21 10:30:44 +00:00
Fix events notifier SQL query (#89)
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
docs

   
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
refactoring

   
testing

   
wontfix

This won't be fixed

No labels
bug
docs
duplicate
enhancement
help wanted
invalid
question
refactoring
testing
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
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
jammella/gancio.notification!89
No description provided.