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

Add ActionController::Caching into api app's document [ci skip] #36060

Merged
merged 1 commit into from Apr 23, 2019

Conversation

st0012
Copy link
Contributor

@st0012 st0012 commented Apr 23, 2019

Rails doesn't support view caching in api controllers by default but the document didn't clearerly declare this nor the manual config needed after including the module manually. So we'll see people get confused like #35602.
This update is suggested in #36038

Rails doesn't support view caching in api controllers by default but the
document didn't clearerly declare this nor the manual config needed
after including the module manually. So we'll see people get confused
like rails#35602.
@rails-bot rails-bot bot added the docs label Apr 23, 2019
@st0012 st0012 changed the title Add ActionController::Caching into api app's document Add ActionController::Caching into api app's document [ci skip] Apr 23, 2019
@st0012
Copy link
Contributor Author

st0012 commented Apr 23, 2019

cc @rafaelfranca

@rafaelfranca rafaelfranca merged commit 3051007 into rails:master Apr 23, 2019
@st0012 st0012 deleted the fix-35602-doc branch April 23, 2019 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants