Repository navigation
Fill gaps from assistant conversations: env var encoding and JSM webhook - #481
Merged
Merged
Conversation
Contributor
Author
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
This was referenced Oct 29, 2026
Merged
hlenke
reviewed
Nov 3, 2026
hlenke
left a comment
Member
There was a problem hiding this comment.
Needs changes · Trim this PR to the JSM note: main already explains triple brackets on the API check page, and the branch conflicts with main on the webhooks heading.
Merge readiness
- Conflict. GitHub reports the PR as
CONFLICTING. Since the merge base (260ea2a),mainrenamed the heading this PR edits from### OpsGenieto### Opsgenie(#734, casing), so the changed heading line conflicts. Rebase onmainand use theOpsgeniespelling (see the inline suggestion). - API check page is already covered on
main. #591 (28738c0) rewrote the sentence atdetect/synthetic-monitoring/api-checks/configuration.mdx:96onmain. It now says that double-bracket variables are HTML-escaped (&becomes&) and that triple brackets insert the raw value, for example for an API key or token. The new#### Preserving special characters with triple bracketssubsection repeats that and adds nothing a reader needs. Drop the whole hunk in this file (inline comment). - Overlap. Open PR #416 also edits
detect/synthetic-monitoring/api-checks/configuration.mdx. Dropping this PR's hunk there removes the overlap. - CI passes (Mintlify, docs-gate, preview-checks, static-docs-checks).
- Cross-file impact. Renaming the heading changes the
#opsgenieanchor. No page onmainlinks towebhooks#opsgenie, so nothing breaks.
Inline comments: 1 important, 1 wording.
What I checked
- Read the full diff
260ea2ae..refs/docs-review/pr-481(2 files, +9/-1) and the PR's version of both pages. - Compared it with
origin/main.configuration.mdx:96already covers the escaping and triple brackets (#591).webhooks.mdx:276is now### Opsgenie(#734).mainalso changed the retry and signature sections ofwebhooks.mdx(DOC-175, #744), which this PR does not touch. gh pr checks 481: all pass.gh pr view 481:mergeable=CONFLICTING,mergeStateStatus=DIRTY.pr_impact.py: only flags/integrations/alerts/webhooksas a hub page. Searchedmainforwebhooks#opsgenie: no inbound links.- "No dedicated JSM alert channel": the alert channel
typeenum inapi-reference/openapi.jsonlists EMAIL, SLACK, SLACK_APP, WEBHOOK, SMS, PAGERDUTY, OPSGENIE and CALL. The checkly-cli constructs at origin/main@4e450744 and the monorepo API contract at origin/main@28c905592a have no JSM or Jira Service Management channel either. - JSM endpoint and auth: Atlassian's integration events API reference (https://developer.atlassian.com/cloud/jira/service-desk-ops/rest/v1/api-group-integration-events/) gives
POST https://api.atlassian.com/jsm/ops/integration/v2/alerts, withAuthorization: GenieKey <apiKey>and Opsgenie-style body fields (message,description,responders,tags, and others). - Could not verify: whether JSM accepts the API key in the URL userinfo (
https://KEY@...), as the Opsgenie example below shows. The suggestion therefore names the header form only. I ran no end-to-end test: the remaining claim is about a third-party API, and testing it would need a JSM account.
hlenke
enabled auto-merge (squash)
November 4, 2026 01:55
# Conflicts: # integrations/alerts/webhooks.mdx
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Small, targeted updates to two existing pages to close documentation gaps surfaced by recent assistant conversations.
Changes
&and other characters can break header values, and when to reach for{{{VAR}}}instead of{{VAR}}. The existing one-line note was easy to miss.Context
Assistant traffic in the last week showed users hitting confusion around HTML-escaped env vars in headers (ampersand breaking API keys) and asking how to send Checkly alerts to JSM Operations now that OpsGenie is being wound down. Neither had a clear landing spot in the docs before.