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

Added docs to visibility API quotas for EDU-4119 #7451

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

GSmithApps
Copy link

@GSmithApps GSmithApps commented Mar 11, 2025

What changed?

Added a note to tell future maintainers that if they edit the visibility API list, they should also update the docs.

Why?

  • Main reason: we have this docs PR that was requested by a user; we're making this code change so the docs don't get out of date
  • Secondary reason: so the OSS team checks the PR ⬆️ to be sure it's right

How did you test it?

I didn't

Potential risks

Nothing

Documentation

The intent of this is to aid docs maintenance

Is hotfix candidate?

No

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