Add ability to pass in offset time when creating alerts via API

[cklenk]

Proposal: Add offset to CreateThresholdCheckAsync and CreateDeadmanCheckAsync for alerting

Current behavior: Unable to set offset time when using CreateThresholdCheckAsync and CreateDeadmanCheckAsync methods

Desired behavior: Pass in optional offset time with functions

Alternatives considered: Manually create alerts

Use case: Ease of development and cleaner solution

[powersj]

Hi,

When you say offset time, what is your use-case for this offset? Is it a delay in notification?

Thanks

[cklenk]

Yes, that is correct. To account for late data that might come in.

Thanks, Cameron

[powersj]

Based on the API docs, it does look like the threshold type check has a offset parameter that is not exposed in this client library.

We can investigate this further, but would also be happy to see a PR with a fix to include this parameter.

Thanks!