-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Fix Exchange manager docs #27024
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
base: master
Are you sure you want to change the base?
Fix Exchange manager docs #27024
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdate the fault-tolerant execution docs by adding the missing File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
|
@ebyhr could you please take a look at this docs update when you have a spare minute? |
|
to ruin the first impression completely I've messed up during the update. Fixing |
8c3db2a to
2e31331
Compare
Also, remove redundant backtick.
47e04df to
a9adfb8
Compare
|
@etolbakov You don't need to rebase unless there is a conflict. Also, please use "rebase" instead of "merge" from next time. |
Description
I've noticed that
exchange.max-page-storage-sizeproperty exists in the code but is not mentioned in the Exchange manager docs.I assume that it's just missing/overlooked. Happy to provide a small fix.
Fixes #27023
Release notes
( x ) This is not user-visible or is docs only, and no release notes are required.
Summary by Sourcery
Documentation: