| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| | |
Testing Guide: unnecessary comment
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This has been around as far back as I can
[see](https://github.com/rails/rails/blob/5137d03cc5b2a5f0820bdcf11b0fffe5bf461470/guides/source/testing.md).
No need to specify the obvious.
[ci skip]
|
| |
| |
| |
| |
| |
| | |
This line refers only to an upgrade situation, but same is need to setup AST in a new application.
[ci skip]
|
|/
|
|
|
|
|
|
|
|
|
| |
Paperclip has officially been deprecated, so we shouldn't mention it
anymore.
CarrierWave could still be referenced, as there are use cases where it
currently makes more sense, but for simplicity, I thought removing the
mention of third party libraries made sense. If we want to talk about
them, listing "alternatives" within the Active Storage guide could make
more sense.
|
|\
| |
| |
| |
| | |
albertoalmagro/albertoalmagro/prefer-rails-command-over-bin-rails
Prefer rails command over bin/rails
|
| |
| |
| |
| | |
This commit substitutes references to rails/rake task for rails command
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This commit integrates most used previously rake commands into the
above outer list. Ordering is based on their individual predicted
frequency of use.
Separation between bin/rails tasks is also removed to display all
commands at the same level.
It also removes references to rake and tasks and substitutes them
for command.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
As discussed in #33203 rails command already looks for, and runs,
bin/rails if it is present.
We were mixing recommendations within guides and USAGE guidelines,
in some files we recommended using rails, in others bin/rails and
in some cases we even had both options mixed together.
|
| |
| |
| |
| |
| | |
http links will be redirected to the https version, but still better to
just directly link to the https version.
|
| | |
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
Modify the expression's issue for `errors.add` document.
[ci skip]
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Since we have `has_secure_token`, it is too confusing to use `_token`
suffix with `has_secure_password`.
Context https://github.com/rails/rails/pull/33307#discussion_r200807185
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
bogdanvlviv/improve-docs-test-of-has_secure_password
Improve docs/test of `has_secure_password`
|
| | |/
| |/|
| | |
| | |
| | | |
`has_secure_password` allows configuring name of attribute since #26764.
This commit adds a mention about it in the Active Model Basics Guide.
|
|\ \ \
| |/ /
|/| |
| | |
| | | |
Specify location for engine-specific commands
[ci skip]
|
| |/
| |
| |
| |
| |
| | |
Be more specific when pointing out where the commands relating
to the engine should be run
[ci skip]
|
|\ \
| |/
|/| |
Adds `Rails::Command::NotesCommand` and makes `rake notes` use it under the hood
|
| |
| |
| |
| |
| |
| | |
* Get rid of references to rake notes in the documentation
* Get rid of references to environement variables used in SourceAnnotationExtractor
* Updates the command line guide to reflect the new rails notes API
|
|\ \
| | |
| | | |
Change location for running copy migrations command
|
| |/
| |
| |
| | |
This fixes the location from where the command to copy migrations from the engine to the application should be run
[ci skip]
|
| | |
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
dikond/explicit_action_mailer_deliver_later_queue_name
Note deliver_later_queue_name gotcha in ActionMailer guide
|
| | | |
|
| | |
| | |
| | |
| | | |
[ci skip]
It adds a note to the Active Storage documentation regarding the use of Mirror Service not being compatible with the use of the Direct Uploads as described later on in the documentation, as described in issue #32732
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
[ci skip] Improved wording by making it clear the form_with
helper method was being referenced in the first paragraph and
that the second paragraph is a continuation of the form_with
train of thought. Additionally, a connection was made to examples
of the form_with usage being described.
|
|/ /
| |
| |
| |
| |
| | |
This list in the menu is currently long, and the order of the items not
immediately obvious. I often find my self rescanning to find what I'm
looking for, so I imagine that others do to.
|
| |
| |
| |
| | |
[Jon Moss & Xavier Noria]
|
| |
| |
| | |
Also fix syntax highlighting in the more advanced JS example.
|
| | |
|
| |
| |
| |
| |
| |
| | |
config.active_support.use_authenticated_message_encryption [ci skip]
related to https://github.com/rails/rails/issues/33170
|
| |
| |
| |
| | |
It is necessary to specify a value to `config.active_storage`.
|
| |
| |
| | |
Since the ability to whitelist arbitrary hashes was added (https://github.com/rails/rails/issues/9454 was resolved by https://github.com/rails/rails/commit/e86524c0c5a26ceec92895c830d1355ae47a7034), this example is no longer outside of what strong_params can do. Moved this specific example out of the "Outside the Scope" section and into the regular "Examples" section, but left the "Outside the Scope" section as it was since the advice is still relevant for weirder whitelisting situations (maybe someone wants to add a new example that can't be handled natively).
|
| |
| |
| |
| | |
[ci skip]
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Remove general sentence that explains that all methods that Active Support
brings to `Date` `Time`, `DateTime` are defined in
`active_support/core_ext/date/calculations.rb` since it isn't true.
Add a references to an appropriate file where methods are defined
like it is done with the other methods in this guide.
Context https://github.com/rails/rails/pull/32552#discussion_r181216246
Related to #32543, #32552
[ci skip]
|
| | |
|
| |
| |
| |
| | |
Add mention about default value of `config.active_model.i18n_full_message`.
|
|\ \
| | |
| | | |
Fix active_model/errors docs [ci skip]
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- Fix indentation.
- Add a missing dot to the end of the sentence.
Related to #32956
|
| | |
| | |
| | |
| | | |
[ci skip]
|
|\ \ \
| | | |
| | | | |
Remove ineffective no-warnings flags from Contribute guide
|
| | | |
| | | |
| | | |
| | | | |
This is not working. [ci skip]
|
|\ \ \ \
| |_|/ /
|/| | | |
Dev-Dep guide: mention AST third party requirements
|
| | | |
| | | |
| | | |
| | | | |
[ci skip]
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Avoid the likes of:
```
Error:
ActiveStorage::RepresentationTest#test_representing_an_MP4_video:
Errno::ENOENT: No such file or directory - ffmpeg
test/models/representation_test.rb:27:in `block in <class:RepresentationTest>'
```
Also note additiona macOS requirements in ActiveStorage guide.
[ci skip]
|