aboutsummaryrefslogtreecommitdiffstats
path: root/guides
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Update The Rails Initialization Process of guide [ci skip]Yoshiyuki Hirano2017-08-201-7/+10
| | | | |
* | | | | Update database config in The Rails Comamnd Line of guide [ci skip]Yoshiyuki Hirano2017-08-201-4/+7
|/ / / /
* | | | Fix doc format in Active Storage Setup [ci skip]Koichi ITO2017-08-191-1/+1
| | | | | | | | | | | | | | | | Follow up of 2ebb284ef5bf2c36bd8ba0a4524cab83810f12ab.
* | | | Use ssl in guide and comment [ci skip]Yoshiyuki Hirano2017-08-198-15/+15
| | | |
* | | | Use https instead of http or git in guide [ci skip]Yoshiyuki Hirano2017-08-191-8/+8
| | | | | | | | | | | | | | | | * Update url in Development Dependencies Install
* | | | Cleanup development_dependencies_install.mdJon Moss2017-08-181-14/+15
| | | | | | | | | | | | | | | | | | | | | | | | Lots of grammar cleanup, and also changing to use macOS. [ci skip]
* | | | Add Yarn installation instructions for Active StorageJon Moss2017-08-181-0/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a requirement when working on the JS portions of ASt. Using npm instead of Yarn is a bad idea here since the lockfile is in Yarn's format. [ci skip]
* | | | Add note about JSON/JSONB serialization changesJon Moss2017-08-171-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The initial commit (efaa6e4f79d457c2cdd08cbc56d63bc972a6993c) that changed this behavior was intended to be a minor change, but ended up becoming a large-ish breaking change within Active Record. This is because instead of only JSON encoding `Hash`es or `Array`s in `#serialize`, we now encode all values passed in. This is an issue if you're passing in a `String`, that has already been transformed from a `Hash` to a `String`, since your data is now being double encoded. Unfortunately, the change was included in one of the v5.0.0 beta releases, and it is too late to revert without huge ripple effects. Thus, all we can do is update the documentation (via this commit), and add some test coverage (coming soon in a PR) for the new behavior. Please note that in the documentation I talk about deserialization, not about serialization, where the actual change occurred. This is because you won't notice any changes in serialized data until you try and deserialize it. Also to make the change itself (confusing until you read through everything multiple times) easier to understand. Related #27788, #25594, #26101, #24234, #28292, #28285, #28285, and probably others. [ci skip]
* | | | Remove obsolete explanationohbarye2017-08-171-1/+1
|/ / / | | | | | | | | | | | | | | | about passing string to `:if` and `:unless`. This is a follow up to https://github.com/rails/rails/pull/28058 [ci skip]
* | | Small grammar fixesJon Moss2017-08-161-1/+1
| | | | | | | | | | | | | | | | | | Changed sentence to be more clear [ci skip]
* | | Change to pluralJon Moss2017-08-161-1/+1
| | | | | | | | | | | | | | | | | | `it` and `has` are for singular, not plural [ci skip]
* | | Remove period from within linksJon Moss2017-08-161-3/+3
| | | | | | | | | | | | | | | | | | Periods should be outside of the <a> tags [ci skip]
* | | Merge pull request #30261 from bogdanvlviv/improve-guides-pluginsRafael França2017-08-151-33/+24
|\ \ \ | | | | | | | | Update guides/source/plugins.md [ci skip]
| * | | Update guides/source/plugins.md [ci skip]bogdanvlviv2017-08-151-33/+24
| | | |
* | | | Add system_test logs to guide [ci skip]Yoshiyuki Hirano2017-08-162-0/+9
|/ / /
* | | Fix guide about command_line [ci skip]Yoshiyuki Hirano2017-08-151-0/+3
| | |
* | | Fix asset pipeline guide [skip ci]Leandro Lourenci2017-08-141-2/+2
| | |
* | | Fix testing guide [skip ci]Yoshiyuki Hirano2017-08-141-2/+4
| | |
* | | Fix debugging rails application [ci skip]Yoshiyuki Hirano2017-08-141-5/+9
| | |
* | | Use frozen string literal in guides/Koichi ITO2017-08-1321-0/+41
| | |
* | | Merge pull request #30060 from bdewater/aj-callback-docsRafael França2017-08-111-22/+30
|\ \ \ | | | | | | | | Improve callback examples in Active Job guide
| * | | Make one line example fit on one line [ci skip]Bart de Water2017-08-091-3/+1
| | | |
| * | | Improve callback examples in Active Job guide [ci skip]Bart de Water2017-08-031-22/+32
| | | | | | | | | | | | | | | | The advice for symbol/block form is taken from the Active Record guides.
* | | | Merge pull request #30186 from yhirano55/fix_generator_example_codeRobin Dupret2017-08-111-2/+2
|\ \ \ \ | | | | | | | | | | Fix generator example code [ci skip]
| * | | | Fix example codes for generators guide [ci skip]Yoshiyuki Hirano2017-08-111-2/+2
| | | | |
* | | | | Update testing.mdYauheni Dakuka2017-08-111-2/+2
|/ / / /
* | | | add missing collection.reload documentation [ci skip]Julia López2017-08-081-2/+24
| | | |
* | | | Merge pull request #30125 from yukideluxe/add-reload-to-associations-docsRafael França2017-08-081-4/+6
|\ \ \ \ | | | | | | | | | | add reload_association to documentation
| * | | | add reload_association to documentation [ci skip]Julia López2017-08-071-4/+6
| | | | |
* | | | | [ci skip] Update action_mailer_basics.mdYauheni Dakuka2017-08-081-1/+1
| | | | |
* | | | | [ci skip] Prefer cookies.encrypted over signed (#30129)Claudio B2017-08-071-1/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some examples and guides we are recommending to use code like: ```ruby verified_user = User.find_by(id: cookies.signed[:user_id]) ``` My suggestion is to use instead: ```ruby verified_user = User.find_by(id: cookies.encrypted[:user_id]) ``` which invites users to prefer the "newer" encrypted cookies over the "legacy" signed cookies.
* | | | Capitalize RakeJon Moss2017-08-061-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Should _probably_ be done across the board with all of our documentation, but going to leave that for another time :) [ci skip]
* | | | Remove extra space between two sentencesJon Moss2017-08-061-1/+1
| | | | | | | | | | | | | | | | [ci skip]
* | | | Replaces the 17 for the actual call to params[:id] that would make the code ↵Mariano Giagante2017-08-061-1/+1
|/ / / | | | | | | | | | work as intended. [ci skip]
* | | Update working_with_javascript_in_rails.mdGuillermo Iguaran2017-08-011-1/+1
| | |
* | | Update working_with_javascript_in_rails.mdGuillermo Iguaran2017-08-011-1/+1
| | |
* | | Change http postgresql.org links to https [ci skip]yuuji.yaginuma2017-07-303-24/+24
| | | | | | | | | | | | | | | It seems that it accepts only HTTPS connections. Ref: https://github.com/postgres/postgres/commit/7f77cbd996855a06fb742ea11adbe55c42b48fe2
* | | Squashed commit of the following:ydakuka2017-07-281-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | commit 4a3d295f3011e771cddead80de7497ca15d15c13 Author: Yauheni Dakuka <yauheni.dakuka@gmail.com> Date: Fri Jul 28 14:31:35 2017 +0300 Update routing.md commit 620a4ce47288e3ef6504290c78f931214968e7e3 Author: Yauheni Dakuka <yauheni.dakuka@gmail.com> Date: Fri Jul 28 14:19:29 2017 +0300 [ci skip] update routing guide
* | | [ci skip] update routing guideYauheni Dakuka2017-07-281-1/+1
| | |
* | | Require arel master in the benchmark templateSean Griffin2017-07-251-0/+1
| | | | | | | | | | | | Pointing at Rails master generally requires pointing at Arel master
* | | [ci skip] Add documentation for after_create_commit and after_update_commit ↵jagdeepsingh2017-07-251-0/+29
| |/ |/| | | | | callbacks
* | Merge pull request #29884 from padi/update_rails_5_upgrade_guideRafael França2017-07-211-0/+10
|\ \ | |/ |/| Updates Rails upgrade guide on `ActionView::Helpers::RecordTagHelper`
| * Updates Rails upgrade guide for 4.2 to 5.0 regarding the removal of ↵Marc Rendl Ignacio2017-07-211-0/+10
| | | | | | | | `ActionView::Helpers::RecordTagHelper`
* | Merge pull request #27662 from nhorton/patch-1Sean Griffin2017-07-171-1/+1
|\ \ | | | | | | Fixing "Adding Member Routes" documentation
| * | Fixing "Adding Member Routes" documentationNoah Horton2017-01-121-1/+1
| | | | | | | | | Previously said the helpers were preview_photo_url and preview_photo_path, i.e. action_resource, when in fact they are resource_action. i.e. photo_preview_path. Fixed.
* | | Merge pull request #29758 from glaucocustodio/patch-1 [ci skip]Sean Griffin2017-07-171-2/+1
|\ \ \ | | | | | | | | Add documentation for class_attribute default option
| * | | Add documentation for class_attribute optionsGlauco Custódio2017-07-171-2/+1
| | | |
* | | | Merge pull request #29782 from y-yagi/follow_up_29699Kasper Timm Hansen2017-07-161-1/+1
|\ \ \ \ | | | | | | | | | | Set `represent_boolean_as_integer` via `configuration`
| * | | | Set `represent_boolean_as_integer` via `configuration`yuuji.yaginuma2017-07-161-1/+1
| | | | |
* | | | | Replace Erubis with Erubi in a part of guide [ci skip]Koichi ITO2017-07-161-1/+1
|/ / / /