aboutsummaryrefslogtreecommitdiffstats
path: root/guides/source
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #33474 from olivierlacan/old-migrationsRichard Schneeman2018-08-111-3/+22
|\ | | | | Document best practices with old migrations
| * Document best practices with old migrationsOlivier Lacan2018-08-031-3/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The copy here is of course up for discussion but it feels like we need to address the issue of old migrations in the Migration guide because other than mentioning the canonical nature of schema.rb/structure.sql or the actual database compared to migration files, it seems like more guidance would help. Here's a sample of the kinds of question people seem to often ask about old Rails migrations: - https://stackoverflow.com/questions/20119391/delete-old-migrations-files-in-a-rails-app - https://www.reddit.com/r/rails/comments/4ayosd/compacting_migrations_files_or_delete_them/ - https://stackoverflow.com/questions/4248682/is-it-a-good-idea-to-purge-old-rails-migration-files - https://stackoverflow.com/questions/707013/is-it-a-good-idea-to-collapse-old-rails-migrations - https://stackoverflow.com/questions/1981777/rails-remove-old-models-with-migrations - https://stackoverflow.com/questions/3343534/rebase-rails-migrations-in-a-long-running-project The common theme seems to be: "I've got old migrations, should I keep them around on an old project?". My personal stance is that as long as migrations run and don't take too long do so, you should keep them around since it allows people working on the Rails project with you to seamlessly upgrade their local development database without having to do a `db:drop db:schema:load` and lose all their seed data. While writing down this suggested new section it felt like I was describing a very cumbersome process that could be address with a rake task like: ```bash rails db:migrate:remove VERSION=20121201123456 ``` It rollback to the version just before `20121201123456`, delete the migration file, and run `db:migrate` to get back to the latest migration. This of course doesn't address a situation when someone would want to delete or merge all migrations prior to a certain date, which is addressed by [squasher](https://github.com/jalkoby/squasher). I'm not sure this is something we want to encourage people to do. Although I feel like with more and more production Rails apps over 5-years old, it's definitely a concern we should address.
* | Merge pull request #33563 from lzap/foreign-key-note-docRichard Schneeman2018-08-111-0/+4
|\ \ | | | | | | ActiveRecord Guide - sqlite3 foreign keys note
| * | ActiveRecord Guide - sqlite3 foreign keys noteLukas Zapletal2018-08-091-0/+4
| | |
* | | Add missing instructions for FreeBSD [ci skip]Robin Dupret2018-08-091-10/+18
|/ / | | | | | | | | | | | | | | The development dependencies installation guides have the installation instructions for FreeBSD in other sections so let's be consistent regarding the dependencies for Active Storage setup. Also fix a few typos.
* | Merge pull request #33554 from ↵Eileen M. Uchitelle2018-08-081-1/+1
|\ \ | | | | | | | | | | | | saveriomiroddi/sav-correct_updated_at_guide_explanation ActiveRecord Basics guide: correct explanation of the `updated_at` logic [ci skip]
| * | ActiveRecord Basics guide: correct explanation of the `updated_at` logic [ci ↵Saverio Miroddi2018-08-081-1/+1
| | | | | | | | | | | | | | | | | | skip] It's misleanding/incorrect to state that `updated_at` is set on updates, since creation != update (and the column is actually set on creation, too).
* | | Add lambdas in conditional validationsAna María Martínez Gómez2018-08-071-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | As `Lambdas` are a type of `Proc`, they can also be used in the `if`/`unless` option of a validation to decide when the validation is executed. Add this case to the guide for clarification. Closes https://github.com/rails/rails/issues/33212
* | | Merge pull request #33507 from jackc/patch-1George Claghorn2018-08-031-1/+1
|\ \ \ | | | | | | | | Fix file upload location recommendation
| * | | Fix file upload location recommendationJack Christensen2018-08-011-1/+1
| | | | | | | | | | | | Going one level downwards from Rails' /public directory would still be inside the public directory and therefore servable by the web server. Files should stored upwards of the public directory.
* | | | Merge pull request #33452 from thetizzo/rails_32_to_40_doc_updateGeorge Claghorn2018-08-031-0/+11
|\ \ \ \ | | | | | | | | | | [Documentation] Mention default HTTP headers in Rails 3.2 to 4.0 upgrade guide
| * | | | Update Rails 3.2 to 4.0 upgrade guide to include a mention and tip for ↵thetizzo2018-08-031-0/+11
| | | | | | | | | | | | | | | | | | | | handling the addition of configurable default HTTP headers. [ci skip]
* | | | | [ci skip] Update adding member routes guide to fix inconsistency with the ↵Ryan Brooks2018-08-031-1/+1
| |/ / / |/| | | | | | | | | | | implementation. Closes #33518
* | | | Revert "Remove label 'work in progress' for AM and AV guides"Xavier Noria2018-07-311-0/+2
| | | |
* | | | Merge pull request #31883 from ↵Richard Schneeman2018-07-311-2/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | bogdanvlviv/remove-label-in-porgress-for-M-V-guides Remove label 'work in progress' for AM and AV guides
| * | | | Remove label 'work in progress' for AM and AV guidesbogdanvlviv2018-02-041-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I've reviewed "Active Model Basics" and "Action View Overview" guides and looks like they have good enough information and don't have errors. This commit removes label 'work in progress' for these guides.
* | | | | [ci skip] Fix the outdated description for `find_each`.Aditya Kapoor2018-07-311-1/+1
| |_|_|/ |/| | |
* | | | Merge pull request #33434 from utilum/remove_testing_guide_commentEileen M. Uchitelle2018-07-271-1/+1
|\ \ \ \ | |_|/ / |/| | | Testing Guide: unnecessary comment
| * | | Testing Guide: unnecessary commentutilum2018-07-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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]
* | | | AST Guide: install and migrate tasks in new apputilum2018-07-261-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | This line refers only to an upgrade situation, but same is need to setup AST in a new application. [ci skip]
* | | | Reference Active Storage instead of third-party libraries in guidePaul McMahon2018-07-261-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | Merge pull request #33229 from ↵Matthew Draper2018-07-2523-263/+260
|\ \ \ | | | | | | | | | | | | | | | | albertoalmagro/albertoalmagro/prefer-rails-command-over-bin-rails Prefer rails command over bin/rails
| * | | Substitute references to task for commandAlberto Almagro2018-07-066-25/+25
| | | | | | | | | | | | | | | | This commit substitutes references to rails/rake task for rails command
| * | | Improve text readabilityAlberto Almagro2018-07-061-1/+1
| | | |
| * | | Homogenize rails commands and former rake tasksAlberto Almagro2018-07-061-58/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | Recommend use of rails over bin/railsAlberto Almagro2018-07-0623-193/+195
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | | Rails guides are now served over httpsPaul McMahon2018-07-2457-80/+80
| | | | | | | | | | | | | | | | | | | | http links will be redirected to the https version, but still better to just directly link to the https version.
* | | | [ci skip] fix typo in Associations guideK. Rodman Mannix2018-07-221-1/+1
| | | |
* | | | Add situation for belongs to association. [ci skip]lanzhiheng2018-07-181-1/+1
| | | |
* | | | Merge pull request #33364 from lanzhiheng/fix-document-in-model-validatorRyuta Kamizono2018-07-161-18/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Modify the expression's issue for `errors.add` document. [ci skip]
| * | | | Remove the invalid usage document about `errors`. [ci skip]lanzhiheng2018-07-161-18/+0
| | | | |
* | | | | Fix document issue in active record callback about `after_touch` hook.lanzhiheng2018-07-151-4/+4
| | | | |
* | | | | Update documentation for #pluck method [ci skip]Jędrek Domański2018-07-141-0/+6
| | | | |
* | | | | 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]
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-16 12:48:15 +0000