Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Don't mount the ActionCable server by default | David Heinemeier Hansson | 2015-12-17 | 2 | -2/+2 |
| | | | | | | Since we require you to enable it on the client-side, let us do the same on the server-side. Then you’re not running an EventMachine unless you need to. | ||||
* | Avoid Action Cable stubs on --skip-action-cable | David Heinemeier Hansson | 2015-12-16 | 3 | -3/+7 |
| | | | | | Still missing the stuff in app/ – we need to basically switch from a * match to specifically picking out what we need. | ||||
* | No longer needed now that we prefill with app/models/application_record.rb | David Heinemeier Hansson | 2015-12-16 | 1 | -1/+0 |
| | |||||
* | Generate all the ApplicationCable stubs by default, like all other ↵ | David Heinemeier Hansson | 2015-12-16 | 5 | -1/+20 |
| | | | | Application* stubs | ||||
* | Saner increments | David Heinemeier Hansson | 2015-12-16 | 1 | -3/+3 |
| | |||||
* | Use puma as the default development server, if available | David Heinemeier Hansson | 2015-12-16 | 1 | -3/+3 |
| | |||||
* | Eager loading is required for Action Cable and we should use the logging too | David Heinemeier Hansson | 2015-12-16 | 1 | -0/+4 |
| | |||||
* | Only mount the ActionCable server if ApplicationCable has been defined | David Heinemeier Hansson | 2015-12-16 | 1 | -1/+1 |
| | |||||
* | By default, mount Action Cable on the router | David Heinemeier Hansson | 2015-12-16 | 1 | -0/+3 |
| | |||||
* | Add default redis config for Action Cable | David Heinemeier Hansson | 2015-12-16 | 2 | -0/+9 |
| | |||||
* | Merge branch 'master' into merge-action-cable | Rafael Mendonça França | 2015-12-16 | 9 | -33/+53 |
|\ | |||||
| * | Use sprockets 3 in the Rails 5 release | Rafael Mendonça França | 2015-12-16 | 1 | -2/+0 |
| | | |||||
| * | Merge pull request #22567 from gsamokovarov/introduce-application-record | Rafael França | 2015-12-16 | 3 | -0/+15 |
| |\ | | | | | | | Introduce ApplicationRecord, an Active Record layer supertype | ||||
| | * | Introduce ApplicationRecord, an Active Record layer supertype | Genadi Samokovarov | 2015-12-16 | 3 | -0/+15 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's pretty common for folks to monkey patch `ActiveRecord::Base` to work around an issue or introduce extra functionality. Instead of shoving even more stuff in `ActiveRecord::Base`, `ApplicationRecord` can hold all those custom work the apps may need. Now, we don't wanna encourage all of the application models to inherit from `ActiveRecord::Base`, but we can encourage all the models that do, to inherit from `ApplicationRecord`. Newly generated applications have `app/models/application_record.rb` present by default. The model generators are smart enough to recognize that newly generated models have to inherit from `ApplicationRecord`, but only if it's present. | ||||
| * | | Update test generators to use ActionDispatch::IntegrationTest | Abdelkader Boudih | 2015-12-15 | 4 | -25/+37 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Rails 5.1 `ActionController::TestCase` will be moved out of Rails into it's own gem. Please use `ActionDispatch::IntegrationTest` going forward. This changes the generators to use `ActionDispatch::IntegrationTest` and the required URL setup (rather than symbols) for each of the controller actions. Updated fix to #22076. | ||||
| * | | Do not default to any Serializer | Santiago Pastorino | 2015-12-15 | 2 | -6/+1 |
| |/ | | | | | | | We will wait until 5.1 to make a decision | ||||
* | | Only include action cable meta tags if it is not skipped | Rafael Mendonça França | 2015-12-16 | 1 | -0/+2 |
| | | |||||
* | | Needless whitespace | David Heinemeier Hansson | 2015-12-16 | 1 | -2/+0 |
| | | |||||
* | | Include the action cable URL meta tag by default | David Heinemeier Hansson | 2015-12-16 | 1 | -1/+3 |
| | | |||||
* | | Assume that the cable server is running in-process by default, offer option ↵ | David Heinemeier Hansson | 2015-12-16 | 1 | -0/+3 |
| | | | | | | | | to change that in config/environments/production.rb | ||||
* | | Fix the engine reference | Rafael Mendonça França | 2015-12-16 | 1 | -1/+1 |
| | | |||||
* | | We are using railtie, not engine | David Heinemeier Hansson | 2015-12-16 | 1 | -1/+1 |
| | | |||||
* | | Merge remote-tracking branch 'origin/master' into merge-action-cable | Rafael Mendonça França | 2015-12-15 | 1 | -2/+2 |
|\| | |||||
| * | Update seeds.rb.tt | Paul Clegg | 2015-12-14 | 1 | -2/+2 |
| | | | | | | | | Rahm Emanuel doesn't deserve to be in Rails. | ||||
* | | Add option to skip Action Cable | Rafael Mendonça França | 2015-12-14 | 2 | -2/+5 |
| | | |||||
* | | Add action_cable to rails/all and application.rb | Rafael Mendonça França | 2015-12-14 | 1 | -0/+1 |
|/ | |||||
* | Revert "Merge pull request #22569 from seuros/remove-test-case" | eileencodes | 2015-12-13 | 3 | -17/+16 |
| | | | | | | | | | This reverts commit 9712a7a12b7f9e4dcef751ceda8a73c3f4beb11f, reversing changes made to 3e65c3d3886336e9145438cdeacaf4ebec6a48b8. Reverting because this caused test failures and while we have a followup branch there is still one failure that happens randomly and isn't straight forward to fix. | ||||
* | Merge pull request #22569 from seuros/remove-test-case | Eileen M. Uchitelle | 2015-12-13 | 3 | -16/+17 |
|\ | | | | | Controller generators should be creating IntegrationTest stubs instea… | ||||
| * | Controller generators should be creating IntegrationTest stubs instead of ↵ | Abdelkader Boudih | 2015-12-13 | 3 | -16/+17 |
| | | | | | | | | ControllerTest | ||||
* | | Merge pull request #22574 from gsamokovarov/scaffold-500 | Kasper Timm Hansen | 2015-12-13 | 2 | -2/+2 |
|\ \ | | | | | | | Prevent a 500 in the default controller scaffold | ||||
| * | | Prevent a 500 in the default controller scaffold | Genadi Samokovarov | 2015-12-13 | 2 | -2/+2 |
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you update a record with no attributes, you'll hit a 500 from a guard in `assign_attributes` raising: ```ruby ArgumentError: When assigning attributes, you must pass a hash as an argument. app/controllers/users_controller.rb:44:in `block in update' app/controllers/users_controller.rb:43:in `update' test/controllers/users_controller_test.rb:37:in `block in <class:UsersControllerTest>' ``` Not a biggie, but may be quite confusing for the folks new to the framework. | ||||
* | | Merge pull request #22573 from gsamokovarov/web-console-3-0 | Kasper Timm Hansen | 2015-12-13 | 1 | -1/+1 |
|\ \ | | | | | | | Update web-console to 3.0.0 | ||||
| * | | Update web-console to 3.0.0 | Genadi Samokovarov | 2015-12-13 | 1 | -1/+1 |
| |/ | | | | | | | | | Web Console 3.0.0 is compatible with Rails 5, while the 2.x.x releases aren't. | ||||
* / | let config.file_watcher be the way to enable the evented file watcher | Xavier Noria | 2015-12-13 | 2 | -4/+4 |
|/ | | | | | | | | | | | Before this commit, the sole presence of the Listen constant enabled the evented file watcher (unless listen resorted to the polling backend). This way, applications may depend on listen for other stuff independently of this feature. Also, allows teams with mixed setups to decide at boot time whether the evented watcher should be enabled for each particular instance. | ||||
* | We don't need to set config.debug_exception_response_format given that :api ↵ | Jorge Bejar | 2015-12-09 | 1 | -6/+0 |
| | | | | is the default value for only API apps | ||||
* | Adjust comment in development.rb template file for app generator | Jorge Bejar | 2015-12-09 | 1 | -3/+0 |
| | |||||
* | Fix indent in generated Rails API env file | Jorge Bejar | 2015-12-09 | 1 | -6/+6 |
| | |||||
* | Add debug_exception_response_format config to configure DebugException | Jorge Bejar | 2015-12-09 | 1 | -0/+8 |
| | | | | behavior | ||||
* | remove `request_forgery_protection` initializer from rails api | yuuji.yaginuma | 2015-12-06 | 1 | -0/+1 |
| | | | | | In rails api, since default is so as not to use the `protect_from_forgery`, `request_forgery_protection` initializer I think that's unnecessary | ||||
* | add `bin/test` script to rails plugin | yuuji.yaginuma | 2015-11-28 | 3 | -3/+14 |
| | |||||
* | let the generated app Gemfile depend on listen 3.0.5 | Xavier Noria | 2015-11-26 | 1 | -1/+1 |
| | | | | See https://github.com/rails/rails/commit/14b20ce9b38314943dcaf73b8dab7508b70ba487. | ||||
* | Merge pull request #22263 from mastahyeti/csrf-origin-check | Rafael França | 2015-11-26 | 1 | -0/+4 |
|\ | | | | | | | | | Add option to verify Origin header in CSRF checks [Jeremy Daer + Rafael Mendonça França] | ||||
| * | Add option to verify Origin header in CSRF checks | Ben Toews | 2015-11-25 | 1 | -0/+4 |
| | | |||||
* | | Remove <br> from scaffold, in favor of using CSS | Sean Collins | 2015-11-25 | 2 | -3/+7 |
| | | |||||
* | | Merge pull request #22370 from yui-knk/remove_blank_line | Andrew White | 2015-11-24 | 1 | -1/+0 |
|\ \ | | | | | | | Remove a blank line from a template for plugin `routes.rb` | ||||
| * | | Remove a blank line from a template for plugin `routes.rb` | yui-knk | 2015-11-22 | 1 | -1/+0 |
| | | | |||||
* | | | Merge pull request #22272 from y-yagi/remove_test_files_in_plugin_gemspec | Eileen M. Uchitelle | 2015-11-23 | 1 | -3/+0 |
|\ \ \ | |/ / |/| | | don't package test files in plugin gemspec | ||||
| * | | don't package test files in plugin gemspec | yuuji.yaginuma | 2015-11-12 | 1 | -3/+0 |
| |/ | | | | | | | | | | | | | | | In rails engine, there is a dummy application under test, the size of the test file is increased. However, there is no need test files for most users, I think it good to have so as not included by default. | ||||
* | | upgrade listen to 3.0.4 | Xavier Noria | 2015-11-08 | 1 | -1/+1 |
| | | | | | | | | 3.0.3 has a bug in OS X. | ||||
* | | move the listen gem in the Gemfile to the development group | Xavier Noria | 2015-11-08 | 1 | -3/+4 |
| | |