aboutsummaryrefslogtreecommitdiffstats
path: root/guides
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | has_secure_password: use `recovery_password` instead of `activation_token`bogdanvlviv2018-07-081-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | Merge pull request #33307 from ↵Ryuta Kamizono2018-07-081-5/+21
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | bogdanvlviv/improve-docs-test-of-has_secure_password Improve docs/test of `has_secure_password`
| * | | | | Update `has_secure_password` info in the guidebogdanvlviv2018-07-061-5/+21
| | |/ / / | |/| | | | | | | | | | | | | | | | | | `has_secure_password` allows configuring name of attribute since #26764. This commit adds a mention about it in the Active Model Basics Guide.
* | | | | Merge pull request #32986 from kevgathuku/patch-1Ryuta Kamizono2018-07-081-0/+3
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | Specify location for engine-specific commands [ci skip]
| * | | | Specify location for engine-specific commandsKevin Ndung'u Gathuku2018-07-041-0/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | Be more specific when pointing out where the commands relating to the engine should be run [ci skip]
* | | | Merge pull request #33220 from anniecodes/notes-commandKasper Timm Hansen2018-07-051-28/+53
|\ \ \ \ | |/ / / |/| | | Adds `Rails::Command::NotesCommand` and makes `rake notes` use it under the hood
| * | | [ci skip] Update documentation related to `rails notes`Annie-Claude Côté2018-07-031-28/+53
| | | | | | | | | | | | | | | | | | | | | | | | * 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
* | | | Merge pull request #32987 from kevgathuku/patch-2Kasper Timm Hansen2018-07-021-1/+1
|\ \ \ \ | | | | | | | | | | Change location for running copy migrations command
| * | | | Change location for running copy migrations commandKevin Ndung'u Gathuku2018-05-251-1/+1
| |/ / / | | | | | | | | | | | | This fixes the location from where the command to copy migrations from the engine to the application should be run [ci skip]
* | | | fix error of a file namesohopro2018-07-021-1/+1
| | | |
* | | | Remove erroneous remove_column option from exampleGeorge Semenov2018-06-291-1/+1
| | | |
* | | | Merge pull request #33238 from ↵Rafael França2018-06-291-0/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | dikond/explicit_action_mailer_deliver_later_queue_name Note deliver_later_queue_name gotcha in ActionMailer guide
| * | | | Note deliver_later_queue_name gotcha in ActionMailer guide [ci skip]dikond2018-06-271-0/+2
| | | | |
* | | | | Incompatibility of Direct Uploads & Mirror ServiceStanislas Boyet2018-06-291-0/+2
| | | | | | | | | | | | | | | | | | | | [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
* | | | | Improve wording for form_with in Getting Started GuideAnthony Crumley2018-06-281-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [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.
* | | | | Break guides menu 'Digging Deeper' section in twoutilum2018-06-281-9/+13
|/ / / / | | | | | | | | | | | | | | | | | | | | 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.
* | | | OS X -> macOS [Closes #30313]Xavier Noria2018-06-231-2/+2
| | | | | | | | | | | | | | | | [Jon Moss & Xavier Noria]
* | | | Active Storage: Explicit form field in basic exampleHenrik Nyh2018-06-221-1/+5
| | | | | | | | | | | | Also fix syntax highlighting in the more advanced JS example.
* | | | Permit configuring the default service URL expiryGeorge Claghorn2018-06-211-0/+7
| | | |
* | | | add documentation for ↵Thomas Dziedzic2018-06-201-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | config.active_support.use_authenticated_message_encryption [ci skip] related to https://github.com/rails/rails/issues/33170
* | | | Fix Active Storage configuration example [ci skip]yuuji.yaginuma2018-06-161-2/+2
| | | | | | | | | | | | | | | | It is necessary to specify a value to `config.active_storage`.
* | | | Update example for whitelisting arbitrary hashesMichael Holroyd2018-06-141-11/+9
| | | | | | | | | | | | 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).
* | | | Add missing documentation about `config.action_mailer.preview_interceptors` ↵yuuji.yaginuma2018-06-141-0/+6
| | | | | | | | | | | | | | | | [ci skip]
* | | | Update "Active Support Core Extensions" Guidebogdanvlviv2018-06-121-76/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 comma to example in guides/source/active_storage_overview.md [ci skip]Matt Sias2018-06-111-1/+1
| | | |
* | | | Add changelog for #32956 [ci skip]bogdanvlviv2018-06-121-1/+1
| | | | | | | | | | | | | | | | Add mention about default value of `config.active_model.i18n_full_message`.
* | | | Merge pull request #33120 from bogdanvlviv/fix-active_model-errors-docsRyuta Kamizono2018-06-121-1/+1
|\ \ \ \ | | | | | | | | | | Fix active_model/errors docs [ci skip]
| * | | | Fix active_model/errors docs [ci skip]bogdanvlviv2018-06-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Fix indentation. - Add a missing dot to the end of the sentence. Related to #32956
* | | | | Remove backticks around proejct namesutilum2018-06-112-5/+5
| | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | Merge pull request #33092 from utilum/warnings_in_contribute_guideRafael França2018-06-111-6/+0
|\ \ \ \ \ | | | | | | | | | | | | Remove ineffective no-warnings flags from Contribute guide
| * | | | | Remove ineffective no-warnings flags from Contribute guideutilum2018-06-081-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | This is not working. [ci skip]
* | | | | | Merge pull request #32997 from utilum/dev_dep_guide_update_ast_requirementsRafael França2018-06-112-5/+34
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Dev-Dep guide: mention AST third party requirements
| * | | | | Update ffmpeg -> FFmpeg, mutools|mupdf -> muPDFutilum2018-06-081-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
| * | | | | Dev-Dep guide: mention AST third party requirementsutilum2018-06-082-5/+33
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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]
* | | | | Merge pull request #32956 from Shopify/i18n_activemodel_errors_full_messageRafael França2018-06-111-0/+4
|\ \ \ \ \ | | | | | | | | | | | | Allow to override the full_message error format
| * | | | | Add global config for config.active_model.i18n_full_messageMartin Larochelle2018-06-051-0/+4
| | | | | |
* | | | | | Update `config.active_storage.variable_content_types` default value in ↵yuuji.yaginuma2018-06-101-1/+1
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | guides [ci skip] Follow up of 3915a47.
* | | | | Fix example of `index_with`bogdanvlviv2018-06-071-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Clarify executor of `public_send`. - Do not wrap `Interval.all_day` into [] since an array is expected as a returned value. Related to #32523. [ci skip]
* | | | | Escape newlines in the GCS private key for valid YAML [ci skip]George Claghorn2018-06-061-1/+1
| | | | |
* | | | | Merge pull request #33047 from artfuldodger/patch-3Eileen M. Uchitelle2018-06-041-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Fix testing guide typo
| * | | | | Fix testing guide typoJon Evans2018-06-011-1/+1
| | | | | |
* | | | | | Correct default value of `unknown_asset_fallback`James Adam2018-06-041-1/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | `Rails.application.config.assets.unknown_asset_fallback` now defaults to `false` in an unmodified Rails application. [ci skip]
* | | | | Merge pull request #33039 from ↵Prem Sichanugrist2018-06-021-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | ybakos/guides/layouts_and_rendering_class_spelling_fix Capitalize class name 'Product' in layouts and rendering guide.
| * | | | | Capitalize class name 'Product'.Yong Bakos2018-06-011-1/+1
| | |/ / / | |/| | | | | | | | | | | | | | | | | | In this context, the phrase 'product instance' refers to the Product class, so capitalize it.
* / | | | Remove 'Current.' prefix from example.Yong Bakos2018-06-011-2/+2
|/ / / / | | | | | | | | | | | | | | | | The prior sentence is referring just to an existing user object, so the sudden magical 'Current.' prefix doesn't make sense. Remove it.
* | | | add CHANGELOG/Docutilum2018-05-221-7/+12
| | | |
* | | | [ci skip] Add documentation/changelog entry.Kasper Timm Hansen2018-05-211-0/+15
| | | |
* | | | Fix markdown [ci skip]Rafael Mendonça França2018-05-171-1/+1
| | | |
* | | | Demonstrate ActiveStorage::Blob#open in the Active Storage guideGeorge Claghorn2018-05-161-42/+9
| | | |
* | | | Added a lot of Oxford commasAnthony Crumley2018-05-1034-115/+115
| | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] A regular expression was used to find a lot of missing Oxford commas and add them. The regular expression was as follows. ", ([a-zA-Z0-9.\`:'\"]+ ){1,6}(or|and) "