-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add initial customer limit mutation docs #18481
Conversation
|
Hi @tpitale 👋 Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days. We will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 10 to 20 minutes). |
✅ Deploy Preview for docs-website-netlify ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tpitale I wasn't able to get the mutation to pass a linter, so I asked ChatGPT for some tips. I also updated the placeholders to SNAKE_CASE per our style guide. How does this look to you?
Hey @rhetoric101 Sorry for the delay, I was out sick for a few days. Let me take a look and make some updates. Thanks! |
And yeah, this is the only mutation for now. But there will be a second one in short order to Otherwise, this looks great, thank you for the updates! Ready to merge when you are. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tpitale thanks for adding this!
Please follow conventional commit standards
in your commit messages and pull request title.
Give us some context
This update adds documentation for a new customer limit mutation specifically for controlling the per-metric cardinality limit.