Skip to content

Conversation

@Dgiordano33
Copy link
Collaborator

Affected Components

  • Content & Marketing
  • Pricing
  • Test
  • Docs
  • Learn
  • Other

Notes for the Reviewer

New Dependency Submission

Screenshots

Copy link
Contributor

@sujaya-sys sujaya-sys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guolau I don't really have context on this change so mainly looked for wording inconsistencies. Hope that helps - feel free to disregard anything that doesn't make sense :)

@@ -1,3 +1,4 @@
---
openapi: get /v1/analytics/api-checks/{id}
title: Get API Check analytics
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
title: Get API Check analytics
title: Get API check analytics


#### Broken Navigation Links (in docs.json but no MDX file)

| Missing File |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm missing context on the purpose of this doc, but why are we documenting issues, missing files, open questions, inconsistencies etc here?

Copy link
Contributor

@guolau guolau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this PR, Dan. IMO will look much better than our current API reference once we've polished it up.

Comment on lines +646 to +656
{
"group": "Triggers",
"pages": [
"api-reference/triggers/get-the-check-group-trigger",
"api-reference/triggers/create-the-check-group-trigger",
"api-reference/triggers/delete-the-check-group-trigger",
"api-reference/triggers/get-the-check-trigger",
"api-reference/triggers/create-the-check-trigger",
"api-reference/triggers/delete-the-check-trigger"
]
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably go in the Detect section, since it's about triggering check runs

"api-reference/checks/update-a-check"
]
},
{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wouldn't separate these endpoints into Uptime and Synthetic since some of these endpoints apply to both (like list all checks).

I think something like this would be bette:

  • api-reference/checks/list-all-checks
  • api-reference/checks/retrieve-a-check
  • api-reference/checks/delete-a-check
  • API check
    • api-reference/checks/create-an-api-check
    • api-reference/checks/update-an-api-check
  • Browser check
    • api-reference/checks/create-a-browser-check
    • api-reference/checks/update-a-browser-check
  • Multistep check
    • api-reference/checks/create-a-multi-step-check
    • api-reference/checks/update-a-multi-step-check
  • DNS monitor
    • api-reference/monitors/create-a-dns-monitor
    • api-reference/monitors/update-a-dns-monitor
  • Heartbeat monitor
    • api-reference/heartbeats/create-a-heartbeat-check
    • api-reference/heartbeats/update-a-heartbeat-check
    • api-reference/heartbeats/get-heartbeat-availability
    • api-reference/heartbeats/get-a-list-of-events-for-a-heartbeat
    • api-reference/heartbeats/get-a-specific-heartbeat-event
  • TCP monitor
    • api-reference/checks/create-a-tcp-check
    • api-reference/checks/update-an-tcp-check
  • URL monitor
    • api-reference/monitors/create-a-url-monitor
    • api-reference/monitors/update-an-url-monitor
  • api-reference/checks/create-a-check
  • api-reference/checks/update-a-check

Comment on lines +716 to +728
"group": "Locations",
"pages": [
"api-reference/location/lists-all-supported-locations",
"api-reference/private-locations/list-all-private-locations",
"api-reference/private-locations/create-a-private-location",
"api-reference/private-locations/retrieve-a-private-location",
"api-reference/private-locations/update-a-private-location",
"api-reference/private-locations/remove-a-private-location",
"api-reference/private-locations/generate-a-new-api-key-for-a-private-location",
"api-reference/private-locations/remove-an-existing-api-key-for-a-private-location",
"api-reference/private-locations/get-private-location-health-metrics-from-a-window-of-time"
]
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like this should go under Platform? Additionally, I'd separate api-reference/location/lists-all-supported-locations (just lists public locations) from the private location endpoints

Comment on lines +732 to +744
"group": "Check Groups",
"pages": [
"api-reference/check-groups/list-all-check-groups",
"api-reference/check-groups/create-a-check-group",
"api-reference/check-groups/retrieve-one-check-in-a-specific-group-with-group-settings-applied",
"api-reference/check-groups/retrieve-a-check-group",
"api-reference/check-groups/update-a-check-group",
"api-reference/check-groups/delete-a-check-group",
"api-reference/check-groups/retrieve-all-checks-in-a-specific-group-with-group-settings-applied",
"api-reference/check-groups/create-a-check-group-v2",
"api-reference/check-groups/update-a-check-group-v2"
]
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably should change the order so that the V2 endpoints are higher up:

Suggested change
"group": "Check Groups",
"pages": [
"api-reference/check-groups/list-all-check-groups",
"api-reference/check-groups/create-a-check-group",
"api-reference/check-groups/retrieve-one-check-in-a-specific-group-with-group-settings-applied",
"api-reference/check-groups/retrieve-a-check-group",
"api-reference/check-groups/update-a-check-group",
"api-reference/check-groups/delete-a-check-group",
"api-reference/check-groups/retrieve-all-checks-in-a-specific-group-with-group-settings-applied",
"api-reference/check-groups/create-a-check-group-v2",
"api-reference/check-groups/update-a-check-group-v2"
]
},
"group": "Check Groups",
"pages": [
"api-reference/check-groups/list-all-check-groups",
"api-reference/check-groups/retrieve-a-check-group",
"api-reference/check-groups/retrieve-one-check-in-a-specific-group-with-group-settings-applied",
"api-reference/check-groups/create-a-check-group-v2",
"api-reference/check-groups/update-a-check-group-v2",
"api-reference/check-groups/delete-a-check-group",
"api-reference/check-groups/retrieve-all-checks-in-a-specific-group-with-group-settings-applied",
"api-reference/check-groups/create-a-check-group",
"api-reference/check-groups/update-a-check-group"
]
},

Comment on lines +798 to +799
"api-reference/status-pages/get-all-subscriptions-for-a-specific-status-page",
"api-reference/status-pages/delete-a-subscription-belonging-to-a-specific-status-page"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd put these in their own "Status Page Subscriptions" category

@@ -0,0 +1,279 @@
# API Reference Documentation Architecture
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, good call to document how the API reference comes together. We can probably clean this up and make it more concise, but this is good for now.

@@ -1,3 +1,4 @@
---
openapi: get /v1/analytics/api-checks/{id}
title: Get API Check analytics
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The title was only changed for API checks. This should be consistent -- either we use this title format for all analytic pages, or we remove this one.

Image

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this endpoint exist publicly? I'm not seeing it in our OpenAPI spec https://api.checklyhq.com/openapi.json

guolau and others added 4 commits February 4, 2026 16:25
Co-authored-by: Susa Tünker <74345218+sujaya-sys@users.noreply.github.com>
Co-authored-by: Susa Tünker <74345218+sujaya-sys@users.noreply.github.com>
Co-authored-by: Susa Tünker <74345218+sujaya-sys@users.noreply.github.com>
…cate.mdx

Co-authored-by: Susa Tünker <74345218+sujaya-sys@users.noreply.github.com>
…s.mdx

Co-authored-by: Susa Tünker <74345218+sujaya-sys@users.noreply.github.com>
guolau and others added 2 commits February 4, 2026 16:26
Co-authored-by: Susa Tünker <74345218+sujaya-sys@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants