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
[ci skip] docs: making clear that perform_caching has a limited impact #20736
Conversation
@@ -8,7 +8,7 @@ module ActionController | |||
# | |||
# You can read more about each approach by clicking the modules below. | |||
# | |||
# Note: To turn off all caching, set | |||
# Note: To turn off all caching provided by ActionController, set |
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.
Action Controller
Thank you for the pull request. Just change ActionController to Action Controller since we are talking about the component, not the module. |
Did the changes and squashed under the same commit message. Should be good now. |
@@ -31,6 +31,8 @@ the relevant `config/environments/*.rb` file: | |||
config.action_controller.perform_caching = true | |||
``` | |||
|
|||
NOTE: Changing the value of `config.action_controller.perform_caching` will only have an effect on the caching provided by the Action Controller component. For instance, it will not impact low-level caching, that we address [below](#low-level-caching). |
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.
Could you wrap this line around 80 chars please ?
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.
This is still not addressed but then this should be good to be merged.
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.
Didn't see that. Since many other places in the same doc are not wrapped (see the info) I didn't see the need for it at first. It's done now.
[ci skip] docs: making clear that perform_caching has a limited impact
Thanks! |
Fix: #20733