fix: clarify comment on SendNotification method regarding SLA linkage

abhinavxd · abhinavxd · commit d1478e1971c6 · 2025-05-26T03:48:41.000+05:30
diff --git a/internal/sla/sla.go b/internal/sla/sla.go
@@ -500,7 +500,7 @@ func (m *Manager) SendNotifications(ctx context.Context) error {
 	}
 }
 
-// SendNotification sends a SLA notification to agents, a schedule notification can be linked to a specific SLA event or applied SLA.
+// SendNotification sends a SLA notification to agents, a schedule notification is always linked to an applied SLA and optionally to a SLA event.
 func (m *Manager) SendNotification(scheduledNotification models.ScheduledSLANotification) error {
 	var (
 		appliedSLA models.AppliedSLA

Original file line number	Diff line number	Diff line change
`@@ -500,7 +500,7 @@ func (m *Manager) SendNotifications(ctx context.Context) error {`
`500`	`500`	`}`
`501`	`501`	`}`
`502`	`502`
`503`		`-// SendNotification sends a SLA notification to agents, a schedule notification can be linked to a specific SLA event or applied SLA.`
	`503`	`+// SendNotification sends a SLA notification to agents, a schedule notification is always linked to an applied SLA and optionally to a SLA event.`
`504`	`504`	`func (m *Manager) SendNotification(scheduledNotification models.ScheduledSLANotification) error {`
`505`	`505`	`var (`
`506`	`506`	`appliedSLA models.AppliedSLA`