Skip to content

APIMGMT-1829: updated rate-limit docs to show client_id instead of access_token#1061

Open
ben-godin-sp wants to merge 3 commits intomainfrom
godin/APIMGMT-1829
Open

APIMGMT-1829: updated rate-limit docs to show client_id instead of access_token#1061
ben-godin-sp wants to merge 3 commits intomainfrom
godin/APIMGMT-1829

Conversation

@ben-godin-sp
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Contributor

@tyler-mairose-sp tyler-mairose-sp left a comment

Choose a reason for hiding this comment

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

Hey @ben-godin-sp, thank you for the proposed changes! A few questions/thoughts.

Does this change affect all API versions? (or is this even a change at all and we have documented this incorrectly in the first place)

This is a public facing document, to the bit about the sp-gateway-routes.yaml I think should be removed since they won't know about that internal file.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 24, 2026

🌎🌎🌎 Visit the preview URL for this PR HERE
built from commit ca04920

@ben-godin-sp
Copy link
Copy Markdown
Author

Hey @ben-godin-sp, thank you for the proposed changes! A few questions/thoughts.

Does this change affect all API versions? (or is this even a change at all and we have documented this incorrectly in the first place)

This is a public facing document, to the bit about the sp-gateway-routes.yaml I think should be removed since they won't know about that internal file.

This is a change that was made a few months ago, and affects all API versions. We published release notes, but I guess we forgot to update these docs. I've removed the links to sp-gateway-routes.yaml and dynamic-rate-limits.yaml.

@github-actions
Copy link
Copy Markdown
Contributor

Slinky Test Report

Last Run: 2026-03-25 14:27:13 CST (Duration: 43.959s)

  • Pass: 2921
  • Fail: 11
  • Total: 2932
  • Files Scanned: 36017

Failures by URL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants