aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Add default redis config for Action CableDavid Heinemeier Hansson2015-12-162-0/+9
| | |
| * | Cargo cult fix to make Celluloid behaveDavid Heinemeier Hansson2015-12-161-0/+3
| | |
| * | Merge branch 'master' into merge-action-cableRafael Mendonça França2015-12-1654-407/+485
| |\ \
| * | | Only include action cable meta tags if it is not skippedRafael Mendonça França2015-12-161-0/+2
| | | |
| * | | Needless whitespaceDavid Heinemeier Hansson2015-12-161-2/+0
| | | |
| * | | Include the action cable URL meta tag by defaultDavid Heinemeier Hansson2015-12-161-1/+3
| | | |
| * | | Require treeDavid Heinemeier Hansson2015-12-161-1/+1
| | | |
| * | | Simpler directory structureDavid Heinemeier Hansson2015-12-161-2/+2
| | | |
| * | | Note that changes to the cable setup requires restarting the serverDavid Heinemeier Hansson2015-12-163-0/+3
| | | |
| * | | @App could have been set elsewhereDavid Heinemeier Hansson2015-12-161-1/+1
| | | |
| * | | Another stab at only mounting redis if the config is thereDavid Heinemeier Hansson2015-12-161-2/+2
| | | |
| * | | Assume that the cable server is running in-process by default, offer option ↵David Heinemeier Hansson2015-12-162-0/+5
| | | | | | | | | | | | | | | | to change that in config/environments/production.rb
| * | | Fix parent classDavid Heinemeier Hansson2015-12-161-1/+1
| | | |
| * | | Fix the engine referenceRafael Mendonça França2015-12-161-1/+1
| | | |
| * | | Doh, should have read ff370ee61754b00f864f3ddd0af62be4dfa0de67David Heinemeier Hansson2015-12-162-2/+2
| | | | | | | | | | | | | | | | Indeed we need this to be an engine for the internal assets to work
| * | | Only initialize redis if the config file is thereDavid Heinemeier Hansson2015-12-161-5/+5
| | | |
| * | | Another bad reference to engine instead of railtieDavid Heinemeier Hansson2015-12-161-1/+1
| | | |
| * | | Include changes made to the engineDavid Heinemeier Hansson2015-12-161-0/+5
| | | |
| * | | Move Cable to ActionCable for client-side constant to avoid conflictsDavid Heinemeier Hansson2015-12-168-49/+49
| | | |
| * | | We are using railtie, not engineDavid Heinemeier Hansson2015-12-163-37/+2
| | | |
| * | | Basic channel generatorDavid Heinemeier Hansson2015-12-157-0/+101
| | | |
| * | | Move the require to the right placeRafael Mendonça França2015-12-152-2/+1
| | | |
| * | | Remove the default loggerRafael Mendonça França2015-12-151-1/+0
| | | | | | | | | | | | | | | | It should be configured through the railtie
| * | | Configure the Action Cable's redis in the engineRafael Mendonça França2015-12-153-23/+7
| | | | | | | | | | | | | | | | This will decouple Action Cable from Rails.
| * | | Accept a Pathname in Application#config_forRafael Mendonça França2015-12-152-1/+20
| | | | | | | | | | | | | | | | That would make possible to use it with action cable configuration.
| * | | Add Action Cable to README.md of Railsclaudiob2015-12-151-1/+3
| | | | | | | | | | | | | | | | [ci skip] Link to actioncable/README.md for more details
| * | | Remove rails/actioncable Travis badgeclaudiob2015-12-151-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] Since Action Cable is getting integrated into rails/rails, the Travis badge for rails/actioncable is not needed anymore. Also following the standard of every other README to have `--` rather than `-` to separating the name of the library from its description.
| * | | Merge remote-tracking branch 'origin/master' into merge-action-cableRafael Mendonça França2015-12-1575-165/+419
| |\ \ \
| * | | | Action Cable need to be a engineRafael Mendonça França2015-12-152-1/+31
| | | | | | | | | | | | | | | | | | | | Otherwise assets will not work
| * | | | Merge pull request #22600 from javan/update-action-cableDavid Heinemeier Hansson2015-12-152-4/+15
| |\ \ \ \ | | | | | | | | | | | | Bring latest changes over from actioncable/master
| | * | | | Bring latest changes over from actioncable/masterJavan Makhmali2015-12-152-4/+15
| |/ / / /
| * | | | Switch from engine to railtieDavid Heinemeier Hansson2015-12-152-6/+6
| | | | |
| * | | | Use a bind param for `LIMIT` and `OFFSET`Sean Griffin2015-12-155-13/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We currently generate an unbounded number of prepared statements when `limit` or `offset` are called with a dynamic argument. This changes `LIMIT` and `OFFSET` to use bind params, eliminating the problem. `Type::Value#hash` needed to be implemented, as it turns out we busted the query cache if the type object used wasn't exactly the same object. This drops support for passing an `Arel::Nodes::SqlLiteral` to `limit`. Doing this relied on AR internals, and was never officially supported usage. Fixes #22250.
| * | | | Deprecate limit strings with commasSean Griffin2015-12-152-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some backends allow `LIMIT 1,2` as a shorthand for `LIMIT 1 OFFSET 2`. Supporting this in Active Record massively complicates using bind parameters for limit and offset, and it's trivially easy to build an invalid SQL query by also calling `offset` on the same `Relation`. This is a niche syntax that is only supported by a few adapters, and can be trivially worked around by calling offset explicitly.
| * | | | Add specific bug reporting guidelines to contributing.md. [ci skip]Tim2015-12-151-7/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add boldface and bullets to make more readable. Make sentences declarative. Remove "you"s.
| * | | | Add option to skip Action CableRafael Mendonça França2015-12-143-2/+10
| | | | |
| * | | | Do not run action cable tests in isolationRafael Mendonça França2015-12-141-0/+1
| | | | | | | | | | | | | | | | | | | | We don't have isolation tests for it yet.
| * | | | Hook in ActionView::Base, not in ApplicationControllerRafael Mendonça França2015-12-141-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The application can have more than one ApplicationController and we want Action Cable helpers in all controllers and their views so we should hook in the same place that others helpers.
| * | | | Require the top level file in the engineRafael Mendonça França2015-12-141-0/+1
| | | | |
| * | | | Add Action Cable tests to travis matrixRafael Mendonça França2015-12-142-0/+2
| | | | |
| * | | | Update Gemfile.lockRafael Mendonça França2015-12-141-5/+0
| | | | |
| * | | | Add action_cable to rails/all and application.rbRafael Mendonça França2015-12-142-9/+11
| | | | |
| * | | | Remove action cable gems that are already in the gemspecRafael Mendonça França2015-12-141-6/+0
| | | | |
| * | | | Remove Alpha disclaimer from ActionCableclaudiob2015-12-141-21/+0
| | | | | | | | | | | | | | | | | | | | [ci skip] Getting to ready to merge into rails/rails
| * | | | Initial stab at adding Action Cable to rails/masterDavid Heinemeier Hansson2015-12-1464-1/+3547
| |\ \ \ \
| | * | | | Get ready to merge into RailsDavid Heinemeier Hansson2015-12-1462-0/+0
| | | | | |
| | * | | | Merge pull request #137 from rails/freshen-client-side-examplesDavid Heinemeier Hansson2015-12-131-23/+47
| | |\ \ \ \ | | | | | | | | | | | | | | Freshen up the client-side subscription examples. Fixes #118
| | | * | | | Associate comments with `perform`Javan Makhmali2015-12-111-2/+2
| | | | | | |
| | | * | | | Freshen up the client-side subscription examples. Fixes #118Javan Makhmali2015-12-111-24/+48
| | | | | | |
| | * | | | | Merge pull request #135 from gregmolnar/readmeDavid Heinemeier Hansson2015-12-131-0/+1
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | add devise example to readme