| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| | |
ActiveStorage guide: Add instruction for test environment
|
| | |
|
| |
| |
| |
| |
| | |
This commit documents how to run all tests or a single test file with a
specific randomization seed.
|
|\ \
| |/
|/| |
Wrap custom id of a header attribute into \" in rails guides
|
| |
| |
| |
| | |
Related to https://github.com/rails/rails/pull/33953
|
|\ \
| |/
|/| |
Fix the LoggerSilence to work as described:
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Following the Rails guide which state that a logger needs to include
the `ActiveSupport::LoggerSilence` as well as
`ActiveSupport::LoggerThreadSafe` modules isn't enough and won't
work.
Here is a test cases with 3 tests that all fails
https://gist.github.com/Edouard-chin/4a72930c2b1eafbbd72a80c66f102010
The problems are the following:
1) The logger needs to call `after_initialize` in order to setup
some instance variables.
2) The silence doesn't actually work because the bare ruby Logger
`add` method checks for the instance variable `@logger`. We need to
override the `add` (like we used to in the ActiveSupport::Logger
class).
3) Calling `debug?` `info?` etc... doesn't work as the bare ruby
methods will check for the instance variable. Again we need to
override this methods (like we used to in the ActiveSupport::Logger
class)
The LoggerSilence won't work without LoggerThreadSafe, but the later
is not public API, the user shouldn't have to include it so I
modified to include it automatically.
Same for the `after_initialize` method. I find unuintitive to have
to call it directly. I modified to instance the variables when the
module get included.
|
|/
|
|
| |
- Followup of https://github.com/rails/rails/pull/34051
|
|\
| |
| | |
Update js docs after the webpacker changes
|
| | |
|
| | |
|
|/
|
|
|
|
|
|
| |
- I found this weird that the LoggerSilence wasn't using the
`ActiveSupport` namespace (AFAIK all other classes have it).
This PR deprecate the use of `LoggerSilence` for
`ActiveSupport::LoggerSilence` instead.
|
| |
|
|
|
|
|
|
| |
Previously the documentation for expires_in suggested that the option
was available for all cache stores. This clarifies that the behavior can
be supported, but is not always supported.
|
|\
| |
| | |
Update guide for validation custom contexts [ci skip]
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The Active Record validations guide's section on custom contexts appears
to be incomplete. the code sample shows a context being added to
validations, but not being used.
Add to the sample code for this section by showing validations being run
with and without the custom context. Add a second sample code block
showing that validations with no context are also run, when a context is
used.
|
|\ \
| | |
| | |
| | |
| | | |
Clarify transactional behavior of after_commit and after_rollback callbacks
[ci skip]
|
| | | |
|
|\ \ \
| | | |
| | | | |
[ci skip] Do not recommend testing stored object in template
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Since `assigns` is not placed by default with Rails, we suggested to
test if the right information was displayed instead of suggesting to
test if the right object was stored in the template.
|
|\ \ \ \
| | | | |
| | | | | |
Fix rails guides markdown.rb and renderer.rb to use custom header id
|
| | | | | |
|
| | | | | |
|
|/ / / /
| | | |
| | | | |
The 'setup' section was misleadingly leaving out :bucket and :region in the sample, but servers don't start without them.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Line items are a holdover from when orders were used in the examples
instead of books.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
ambigity -> ambiguity
Related to #33891
|
|\ \ \ \
| | | | |
| | | | | |
Contributing to Ruby on Rails Guide: standardize git remote names [ci skip].
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Update MySQL server version and character set in Rails Guide [skip ci]
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Related to #33853
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
Follow up #33751
|
| |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Add mention about it to the "Active Support Instrumentation" guide
Related to https://github.com/rails/rails/pull/33751#discussion_r214745153
Follow up #33751
|
| | | | | |
|
|/ / / / |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Applications can configure the route prefix prepended to the Active
Storage routes. By default this maintains the previous prefix
`/rails/active_storage` but supports custom prefixes.
Before this change the route for serving blobs is fixed to
`/rails/active_storage/blobs/:signed_id/*filename`. After this change
it's possible to configure the route to something like
`/files/blobs/:signed_id/*filename`.
|
| |/ /
|/| | |
|
| | |
| | |
| | |
| | |
| | | |
Add mention that `config.filter_parameters` also filters out sensitive
values of database columns when call `#inspect` since #33756.
|
| | |
| | |
| | |
| | | |
Add backticks to code
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The name of the minitest library is spelled that way: regular font, and
lowercase. Lowercase is used even at the beginning of sentences, see
http://docs.seattlerb.org/minitest/
I double-checked this with @zenspider too (thanks!).
|
|\ \ \
| |/ /
|/| | |
Skip delivery notification when perform_deliveries is false.
|
| | | |
|
| | | |
|
|/ /
| |
| |
| |
| | |
See `ConnectionUrlResolver#database_from_path` in
`activerecord/lib/active_record/connection_adapters/connection_specification.rb`
|
| | |
|
| |
| |
| |
| | |
Closes #32885.
|
|\ \
| | |
| | | |
Use Turbolinks in Rails guides
|
| | | |
|
| | | |
|
|\ \ \
| | | |
| | | | |
Add hooks to ActiveJob around retries and discards
|
| | | | |
|