Update constants, vars, whatever to note that notification delays apply regardless of attempt count

[atc0005]

While working on atc0005/brick#4, I was again reminded of how I ended up implementing notification delays:

// Delay between message submission attempts; this will *always*
// delay, regardless of whether the attempt is the first one or not

This comment above this portion of the select block:

bounce/cmd/bounce/message.go

Lines 244 to 272 in cfbe63e

	// Delay between message submission attempts; this will *always*
	// delay, regardless of whether the attempt is the first one or not
	case <-notificationDelayTimer.C:
	log.Debugf("sendMessage: Waited %v before notification attempt at %v",
	notificationDelay,
	time.Now().Format("15:04:05"),
	)
	ctxExpires, ctxExpired := ctx.Deadline()
	if ctxExpired {
	log.Debugf("sendMessage: WaitTimeout context expires at: %v", ctxExpires.Format("15:04:05"))
	}
	// check to see if context has expired during our delay
	if ctx.Err() != nil {
	msg := NotifyResult{
	Val: fmt.Sprintf(
	"sendMessage: context expired or cancelled at %v: %v, attempting to abort message submission",
	time.Now().Format("15:04:05"),
	ctx.Err().Error(),
	),
	Success: false,
	}
	log.Debug(msg.Val)
	return msg
	}

prompted me to quickly stub in this FIXME entry that I later removed for inclusion into this GH issue:

// FIXME: Need to update wording of constant, docs, etc to emphasize that
// the delay is enforced regardless of whether the notification attempt is
// the first, or a retry

Since I opted to apply the delay regardless of the delivery attempt, I should probably "unhook" the "retries" bit from the associated variables/constants associated with the delay. For example, it is in the function parameter list:

bounce/cmd/bounce/message.go

Lines 194 to 202 in cfbe63e

	// define function/wrapper for sending details to Microsoft Teams
	func sendMessage(
	ctx context.Context,
	webhookURL string,
	msgCard goteamsnotify.MessageCard,
	schedule time.Time,
	retries int,
	retriesDelay int,
	) NotifyResult {

and is also used in the exported SendMessage() function from the atc0005/send2teams project:

bounce/cmd/bounce/message.go

Lines 274 to 276 in cfbe63e

	// Submit message card, retry submission if needed up to specified number
	// of retry attempts.
	if err := send2teams.SendMessage(ctx, webhookURL, msgCard, retries, retriesDelay); err != nil {

As of this writing I've not reviewed that project yet to see if it has similar artifacts from the early design/implementation work, but it's likely.