Skip to content

chore: removes go and python mgmt sdks#1455

Merged
MikeCarbone merged 3 commits into
mainfrom
remove-mgmt-sdks
May 27, 2026
Merged

chore: removes go and python mgmt sdks#1455
MikeCarbone merged 3 commits into
mainfrom
remove-mgmt-sdks

Conversation

@MikeCarbone
Copy link
Copy Markdown
Member

Description

Our build pipeline hasn't been executing these for a while. Let's remove from docs since our support isn't 100%.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment May 27, 2026 4:10pm

Request Review

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Risk LOW: Removes two SDK card entries (Go and Python management API SDKs) from a single content file.

Reasons

  • Only 1 MDX content file modified (content/developer-tools/sdks.mdx)
  • Pure content deletion (14 lines removed, 0 additions)
  • No component, layout, library, or configuration files touched
  • No new pages, sidebar changes, or structural modifications
  • Small, straightforward diff with no logic changes

Notes

  • Verify that the removed SDK links (knock-mgmt-python, knock-mgmt-go) are not referenced elsewhere in the docs
  • Confirm the remaining Node.js/TypeScript Management API SDK card still renders correctly in the SDK card group
Open in Web View Automation 

Sent by Cursor Automation: Docs PR classifier

Copy link
Copy Markdown
Member

@cellomatt cellomatt left a comment

Choose a reason for hiding this comment

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

Added a couple comments, we probably want to tweak the pre-list language slightly. LGTM otherwise!

Comment thread content/__mapi-reference/content.mdx Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
Knock offers an SDK for the management API:

Comment thread content/developer-tools/sdks.mdx Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
Knock also provides an SDK for the [Management API](/developer-tools/management-api), which you can use to programmatically manage your Knock dashboard resources.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
Knock provides a Node.js SDK for the management API. You can also use the [CLI](/developer-tools/knock-cli) for easily interacting with the resources in your Knock account.

@MikeCarbone MikeCarbone merged commit 9b27124 into main May 27, 2026
5 checks passed
@MikeCarbone MikeCarbone deleted the remove-mgmt-sdks branch May 27, 2026 16:14
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.

2 participants