aboutsummaryrefslogtreecommitdiffstats
path: root/guides
Commit message (Collapse)AuthorAgeFilesLines
* Fix punctuation [ci skip]Markov Alexey2015-10-231-1/+1
|
* API guidelines reword re the Oxford comma [ci skip]Xavier Noria2015-10-221-3/+4
|
* Revert "[ci skip] removed extra comma"Xavier Noria2015-10-221-1/+1
| | | | | | | | Reason: That one is an Oxford comma, as per our guidelines. See: http://edgeguides.rubyonrails.org/api_documentation_guidelines.html#comma This reverts commit f184cd34f9ef35ddf85a23c2fb02cc05d3fb26b3.
* [ci skip] removed extra commahemali jain2015-10-231-1/+1
|
* Fix minor typo from #20038Sean Griffin2015-10-201-2/+2
| | | | We use one period after spaces, not two.
* Merge pull request #20038 from imanel/numeric_prependSean Griffin2015-10-201-11/+3
|\ | | | | Use Module.prepend instead of alias_method and unify behavior of all Numeric extensions
| * Unify behavior of all Numeric extensions and use Module.prepend instead of ↵Bernard Potocki2015-05-281-11/+3
| | | | | | | | alias_method
* | Update #20768 to address feedbackSean Griffin2015-10-201-3/+3
| |
* | [Rails Guides] Clarify `belongs_to_required_by_default` optionVladimir Rybas2015-10-201-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | Original: * `config.active_record.belongs_to_required_by_default` is a boolean value and controls whether `belongs_to` association is required by default. Modified version: * `config.active_record.belongs_to_required_by_default` is a boolean value and controls whether validation error is triggered by default if `belongs_to` association is not present or not valid.
* | Merge pull request #20795 from benjiwheeler/validation_guide_ret_valSean Griffin2015-10-201-7/+15
|\ \ | | | | | | activerecord guide: clarified roles of valid method and errors object [ci skip]
| * | activerecord guide: clarified roles of valid method and errors object [ci skip]Ben Wheeler2015-07-081-7/+15
| | |
* | | [ci skip] Replace "destroy` method" with `destroy` action`yui-knk2015-10-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sure this is `destroy` method of PhotosController, but in this chapter these methods which mapped by the router are called "action". For example: ```ruby get '/patients/:id', to: 'patients#show' ``` is described to dispatch "controller's `show` action".
* | | added Rails 4 context for migrations:install taskSimon H2015-10-191-1/+1
| | |
* | | [ci skip] Improve readability in active model basics guideTim Wade2015-10-181-7/+7
| | | | | | | | | | | | | | | * adds/removes a few words * removes an unnecessary comma
* | | [ci skip] Add backend queue starting documentationschneems2015-10-161-3/+12
| | | | | | | | | | | | If you can't start your queue, you won't process much. This change adds external links to the Queue backends that have Active Job specific docs.
* | | Update active_record_migrations.mdAnna Ershova2015-10-151-1/+1
| | | | | | | | | Add 'db/'; corrects structure.rb to structure.sql
* | | Clarifies db can be set up from structure.sql alsoAnnaErshova2015-10-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | I added that *structure.sql* file can be used when *db:reset* is run. *db:reset* tasks states *db:reset* loads database from *db/schema.rb* or *db/structure.sql* depending on the configuration (although *db/schema.rb* is the default), hence the change.
* | | Fix formatting of ActiveRecord PostgreSQL guide.Christian Wesselhoeft2015-10-131-4/+6
| | |
* | | [ci skip] Fix typo in deep_dup docsEric Turner2015-10-121-1/+1
| | |
* | | Merge pull request #21906 from scottgonzalez/form-helpersClaudio B.2015-10-081-6/+8
|\ \ \ | | | | | | | | Minor cleanup for form helpers guide
| * | | Minor cleanup for form helpers guide [ci skip]Scott González2015-10-081-6/+8
| | | |
* | | | Improve readability in CSRF section of guideAndy Lampert2015-10-071-4/+3
| | | |
* | | | Revert "Update asset_pipeline.md"Rafael Mendonça França2015-10-071-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit afe672185d5eba95f387c46f855ae7a21ea69fa5. Reason: This change was done by mistake. This option is at the `config` object not in the `config.assets`. See https://github.com/rails/rails/blob/cfb371e96a2789790359ec1db779df490fd253d6/railties/lib/rails/application/configuration.rb#L16
* | | Fix "Esperanto" nameMiguel Parramon2015-10-071-1/+1
| | | | | | | | | | | | | | | Correct spelling is "Esperanto": https://en.wikipedia.org/wiki/Esperanto
* | | Merge pull request #21899 from scottgonzalez/layouts-and-rendering-typoRichard Schneeman2015-10-071-1/+1
|\ \ \ | | | | | | | | Fix typo in docs
| * | | Fix typo in docs [ci skip]Scott González2015-10-071-1/+1
| | | |
* | | | Fix pluralization in docs [ci skip]Scott González2015-10-071-1/+1
|/ / /
* | | [ci skip] Change 'an URL' to 'a URL' as URL doesn't have a vowel soundtanmay30112015-10-064-5/+5
| | |
* | | Update asset_pipeline.mdkentendo2015-10-051-1/+1
| | |
* | | Merge pull request #21796 from dv/docs-change-to-parse-nested-queryEileen M. Uchitelle2015-10-051-7/+0
|\ \ \ | | | | | | | | Change "parse_query" to "parse_nested_query"
| * | | Remove TIP on parse_queryDavid Verhasselt2015-10-051-7/+0
| | | | | | | | | | | | | | | | [ci skip]
* | | | concatenate @lang if @lang is presence to avoid useless slashSeiei Miyagi2015-10-051-1/+1
| | | |
* | | | Add ability to translate rails guides documents.yamlSeiei Miyagi2015-10-052-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some translations of rails guides has their own documents.yaml, for example Japanese: https://github.com/yasslab/railsguides.jp/blob/japanese/guides/source/documents.yaml Traditional Chinese: https://github.com/docrails-tw/guides/blob/master/rails_guides/helpers.rb#L21,L27 https://github.com/docrails-tw/guides/blob/master/source/documents_zh-TW.yaml The [Translating Rails Guides](https://github.com/rails/docrails/wiki/translating-rails-guides) says > Copy the contents of guides/source into your own language directory and translate them. So that sounds ok to copy guides/source/documents.yaml into language specific directly and translating them like other markdown files.
* | | | Merge pull request #20981 from dkoprov/ar_postgres_enum_docs_additionSean Griffin2015-10-041-5/+41
|\ \ \ \ | | | | | | | | | | Add more info on working with postgres ENUM type
| * | | | adds more info on working with postgres ENUM typeDmitry Koprov2015-07-231-5/+41
| | | | |
* | | | | Merge pull request #21857 from cllns/fix-myspace-example-wordingClaudio B.2015-10-041-5/+3
|\ \ \ \ \ | | | | | | | | | | | | Update text on CSS Injection
| * | | | | Update text on CSS Injection / MyspaceSean Collins2015-10-031-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | [skip ci]
* | | | | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2015-10-042-2/+2
|\ \ \ \ \ \
| * | | | | | [ci skip] Add `:` before code blocks.yui-knk2015-09-251-1/+1
| | | | | | |
| * | | | | | Add a missing period to layouts_and_rendering.md [ci skip]Juanito Fatas2015-09-241-1/+1
| | | | | | |
* | | | | | | Revert "removing Rack::Runtime from the default stack."Aaron Patterson2015-10-033-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 37423e4ff883ad5584bab983aceb4b2b759a1fd8. Jeremy is right that we shouldn't remove this. The fact is that many engines are depending on this middleware to be in the default stack. This ties our hands and forces us to keep the middleware in the stack so that engines will work. To be extremely clear, I think this is another smell of "the rack stack" that we have in place. When manipulating middleware, we should have meaningful names for places in the req / res lifecycle **not** have engines depend on a particular constant be in a particular place in the stack. This is a weakness of the API that we have to figure out a way to address before removing the constant. As far as timing attacks are concerned, we can reduce the granularity such that it isn't useful information for hackers, but is still useful for developers.
* | | | | | | Fixed deprecated string to specify the middleware [ci skip]amitkumarsuroliya2015-10-031-1/+1
| |/ / / / / |/| | | | |
* | | | | | Merge pull request #21851 from y-yagi/use_class_to_specify_middlewareKasper Timm Hansen2015-10-031-5/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | use class to specify the middleware [ci skip]
| * | | | | | use class to specify the middleware [ci skip]yuuji.yaginuma2015-10-031-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | using string for middleware class names is deprecated in 83b767cef90abfc4c2ee9f4b451b0215501fae9a
* | | | | | | removing Rack::Runtime from the default stack.Aaron Patterson2015-10-023-3/+1
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The runtime header is a potential target for timing attacks since it returns the amount of time spent on the server (eliminating network speed). Total time is also not accurate for streaming responses. The middleware can be added back via: ```ruby config.middleware.ues ::Rack::Runtime ```
* | | | | | Refactor AS::Callbacks halt config and fix the documentationRoque Pinel2015-10-012-12/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move from `AS::Callbacks::CallbackChain.halt_and_display_warning_on_return_false` to `AS::Callbacks.halt_and_display_warning_on_return_false` base on [this discussion](https://github.com/rails/rails/pull/21218#discussion_r39354580) Fix the documentation broken by 0a120a818d413c64ff9867125f0b03788fc306f8
* | | | | | Fix documentation for has_many dependant options.Niall Burkley2015-09-291-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * A `has_many` dependant association accepts `[:destroy, :delete_all, :nullify, :restrict_with_error, :restrict_with_exception]` as options. Currently the documentation references `delete` instead of `delete_all` * Adds documentation for other options
* | | | | | Make fuzzy sentence more clearDavid Verhasselt2015-09-281-1/+1
| |_|/ / / |/| | | | | | | | | | | | | | [ci skip]
* | | | | Merge pull request #21738 from cllns/remove-to_s-from-exampleEileen M. Uchitelle2015-09-251-19/+1
|\ \ \ \ \ | | | | | | | | | | | | [Engines Guide] Remove to_s example, since it's outside the scope
| * | | | | Remove to_s exampleSean Collins2015-09-241-19/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's outside the scope of the Engines guide [skip ci]