Update user_and_ip_rate_limits.md to fix typo
What does this MR do and why?
This MR corrects what looks like a typo in the rate limit docs where the previous description could not make sense mathematically :)
Screenshots or screen recordings
not relevant
How to set up and validate locally
not relevant
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Merge request reports
Activity
Hi
@gl-docsteam
, please review this documentation Merge Request.Please also consider updating the
CODEOWNERS
file in the https://gitlab.com/gitlab-org/gitlab project.Edited by Kati Paizeeadded documentation twtriaged labels
Thank you for your contribution to GitLab. We believe that everyone can contribute and contributions like yours are what make GitLab great!
- Our Merge Request Coaches will ensure your contribution is reviewed in a timely manner*.
- If you haven't, please set up a
DANGER_GITLAB_API_TOKEN
. - You can comment
@gitlab-bot label ~"group::"
to add a group label. - If you need help moving the MR forward or finding a reviewer, feel free to ask
@gitlab-bot help
or ping a Merge Request Coach. - Read more on how to get help.
- You can provide feedback on the GitLab Contributor Experience in this survey
This message was generated automatically. You're welcome to improve it.
added Community contribution label
assigned to @LaurentS
- Resolved by Kati Paizee
Hi @LaurentS - thanks for opening this MR! You're right, this does look like a typo.
I think your fix is correct, but I'll ask @qmnguyen0711 to take a quick look to be absolutely sure!
added Technical Writing docsimprovement twdoing typemaintenance labels and removed twtriaged label
requested review from @kpaizee and @qmnguyen0711
changed milestone to %14.9
1 Warning f5968db5: The commit body should not contain more than 72 characters per line. For more information, take a look at our Commit message guidelines. 2 Messages CHANGELOG missing: If you want to create a changelog entry for GitLab FOSS, add the
Changelog
trailer to the commit message you want to add to the changelog.If you want to create a changelog entry for GitLab EE, also add the
EE: true
trailer to your commit message.If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.
This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge. If needed, you can retry the
danger-review
job that generated this comment.Documentation review
The following files require a review from a technical writer:
doc/user/admin_area/settings/user_and_ip_rate_limits.md
The review does not need to block merging this merge request. See the:
-
Metadata for the
*.md
files that you've changed. The first few lines of each*.md
file identify the stage and group most closely associated with your docs change. - The Technical Writer assigned for that stage and group.
- Documentation workflows for information on when to assign a merge request for review.
Generated by
Dangerenabled an automatic merge when the pipeline for 1a657cfd succeeds
Hi @LaurentS,
We would love to know how you found your code review experience in this merge request! Please leave a
or a on this comment to describe your experience.Once done, please comment
@gitlab-bot feedback
below and feel free to leave any additional feedback you have in the same comment.You can also fill out a 5 minute survey to provided additional feedback on how GitLab can improve the contributor experience.
Thanks for your help!
mentioned in commit 3f0faea1
@gitlab-bot feedback wow, that was super quick! I guess the only improvement I can think of would be to provide some estimate of when the result will be visible in the live docs (or is this what the milestone implies?)
added workflowstaging-canary label
added workflowstaging label and removed workflowstaging-canary label
added workflowcanary label and removed workflowstaging label
added workflowproduction label and removed workflowcanary label
added releasedcandidate label
added releasedpublished label and removed releasedcandidate label