Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DEVDOCS-6211 - update concurrent call limit (category trees) #799

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bc-terra
Copy link
Contributor

@bc-terra bc-terra commented Feb 4, 2025

Added messaging about concurrent call rate limit to Get a Category Tree endpoint. This will be affected by an experiment in the next 90 days.

DEVDOCS-6211

What changed?

  • The Get a Category Tree endpoint is getting a concurrent call limit to prevent overloading on the back-end.
  • This endpoint is now documented.

Release notes draft

  • Updated Get a Category Tree endpoint to include mention of the default concurrent call limit.

Anything else?

ping {names}

Added messaging about concurrent call rate limit to Get a Category Tree endpoint. This will be affected by an experiment in the next 90 days.
@bc-terra bc-terra requested a review from a team as a code owner February 4, 2025 16:01
@bc-terra bc-terra added ready for review This PR is complete and awaiting a review. awaiting-css-cue labels Feb 4, 2025
@jyuricle
Copy link
Contributor

jyuricle commented Feb 5, 2025

We should put in on hold until all the blockers are addressed

@bc-terra bc-terra removed the ready for review This PR is complete and awaiting a review. label Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants