| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
I've noticed that once `<h1>Editing article</h1>` L1157 was the title and then later on it's `<h1>Edit article</h1>` L1358 which kind of makes more sense...
[ci skip]
|
| |
|
|\
| |
| |
| |
| | |
malclocke/specify_ar_message_interpolation_names_in_i18n_guide
List names of error interpolation variables.
|
| | |
|
|\ \
| | |
| | | |
Clarify migration to ApplicationRecord in upgrade guides [ci skip]
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Style/SpaceBeforeBlockBraces
Style/SpaceInsideBlockBraces
Style/SpaceInsideHashLiteralBraces
Fix all violations in the repository.
|
|/ / |
|
| | |
|
| |
| |
| |
| |
| |
| | |
Reason: See https://github.com/rails/rails/pull/26163#issuecomment-239703322
This reverts commit 39effc857e5c774670d6fad1a26aebcc33c51f0a.
|
|\ \
| | |
| | | |
Clarify and fix typos in autoloading disabled section
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The Guides section about autoloading being disabled was slightly confusing
(#24724) and didn't directly reference the removed feature by name
(config.autoload_paths) making it much harder for someone to search the upgrade
guides for a mention or serendipitously find it via a Google search when running
into autoloading issues.
I also fixed some confusing turns of phrase and a missing word.
/cc @vipulnsward @jvanbaarsen
|
|/ /
| |
| |
| | |
- It is Action Controller config, not Action View.
|
| | |
|
| | |
|
|\ \
| | |
| | | |
Minor doc fix related to ActiveModel::SecurePassword [ci skip]
|
| | | |
|
| | | |
|
|/ /
| |
| |
| | |
Hash syntax auto-correcting breaks alignments. 411ccbdab2608c62aabdb320d52cb02d446bb39c
|
|\ \
| | |
| | | |
Document know limitation about using `references` in conjunction with custom select clauses [ci skip]
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
loading causing it to ignore custom select clauses.
[ci skip]
Fixes #24314
|
|\ \ \
| |/ /
|/| | |
Add link to 'Pry' in debugging guide [ci skip]
|
| | | |
|
| | |
| | |
| | |
| | | |
Follow up to fe859a54219740fa8b4e09a592820d2ee12ba222
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Implement naive partial caching mechanism.
Add test for LogSubscriber
Use ActionView::Base#log_payload to store log_subscriber's payload, so we can pass cache result into it.
Fixed tests
Remove useless settings
Check if #log_payload exists before calling it. Because other classes also includes CacheHelper but don't have is attribute
Use @log_payload_for_partial_reder instead of #log_payload to carry ActionView's payload.
Update test's hash syntax
Add configuration to enable/disable fragment caching logging
Remove unless test and add new test to ensure cache info won't effect next rendering's log
Move :enable_fragment_cache_logging config from ActionView to ActionPack
Apply new config to tests
Update actionview's changelog
Update configuration guide
Improve actionview's changelog
Refactor PartialRenderer#render and log tests
Mute subscriber's log instead of disabling instrumentation.
Fix typo, remove useless comment and use new hash syntax
Improve actionpack's log_subscriber test
Fix rebase mistake
Apply new config to all caching intstrument actions
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
The current code base is not uniform. After some discussion,
we have chosen to go with double quotes by default.
|
|\ \
| | |
| | | |
Correct usage for acceptance validator in guide
|
| | | |
|
| | | |
|
| |/
|/|
| |
| |
| |
| |
| |
| |
| |
| | |
Due to the limited support for inotify (Microsoft/BashOnWindows#216) in
WSL the best thing to do at moment is add a note about skipping spring
and listen when generating your application in the Getting Started guide.
[skip ci]
Fixes #26054.
|
| |
| |
| |
| |
| |
| |
| | |
Some Rails core members agreed to continue to maintain 4.2 until 5.1 is
released.
[ci skip]
|
|\ \
| |/
|/| |
|
| |
| |
| |
| | |
That file has been removed and replaced with new_framework_defaults.rb
|
| |
| |
| |
| | |
`_key_[]` [ci skip]
|
| | |
|
| |
| |
| |
| |
| |
| | |
guides [ci skip]
- Followup of #25692.
|
| | |
|
|\ \
| | |
| | | |
[ci skip] Add documentation for error using select with order
|
| | | |
|
|/ /
| |
| |
| |
| |
| | |
ActionController::UnknownFromat in ArticlesController#new
Restored texts about XML and JS template handlers. [ci skip]
|
|\ \
| | |
| | | |
Add proper description for `Rails.cache.fetch` [ci skip]
|
| | | |
|
|\ \ \
| | | |
| | | | |
Explain meaning of the code very well and Remove confusion. [ci skip]
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
Code like `render(topics) => render("topics/topic")` adds confusion
for the readers. On first impression, that code feels like hash kind
of data, but it's not. So make it more clear and meaningful.
|
|/ /
| |
| |
| |
| |
| | |
See the rationale in the documentation included in this patch.
We are going to gradually introduce this predicate in the code base.
|
|\ \
| | |
| | |
| | | |
Setup default session store internally, no longer through an initializer
|