aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Header TypoDavid Kuhta2016-02-231-1/+1
|
* `stream_for` exampleDavid Kuhta2016-02-231-0/+15
|
* Explicitly denote that channel.rb is a default file.David Kuhta2016-02-231-2/+2
|
* Change ActionCable.server.broadcast to XChannel.broadcast_toDavid Kuhta2016-02-231-10/+7
| | | Using broadcast directly off server is not recommended
* Create action_cable_overview.md resolves #23176 [ci skip]David Kuhta2016-02-141-0/+659
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rails Guide for Action Cable Added content from README Pull additional information from Action Cable README and restructured Client/Server elements Update to list numbering Switched from list numbers to headings in the examples Change AppearancesChannel to AppearanceChannel Word missing fixed missing word. Removed Mistaken nesting of Channel class Removed nesting of ChatChannel and AppearanceChannel from within ApplicationChannel Module. (Copy&Paste error) Incorporated first round of comments 1. Capitalize "action cable" 2. Separated "Consumers require... via JavaScript" into two sentences 3. Minor typographical correction (remember...) 4. Extra backtick 5. Revised text which implied Redis was the exclusive storage adapter to reflect it's position as default. 6. Revised reference to denote correct config file location: config/cable.yml 7. Added adapter: redis to environment configuration blocks 8. Capitalized "R" for "Rack" 9. Revised syntax for routing to reflect the hash syntax. (to: NOT :to =>) 10. Removed reference to Action Cable being separate from Rails. 11. Began revision for adapter API [I believe this requires a reformatting of the 'Redis' portion of the configuration section to simply "Storage Adapters"] 12. Celluloid -> Concurrent-ruby Moved errant grave mark [ci skip] Reordered "In App" above "Standalone" [ci skip] Reordered to reflect "In App" as preferable to "Standalone" Action Cable Deployment [ci skip] Removed paragraph that alludes to Action Cable not being able to run in the same process as Rails. Removed EM reference and updated deployment [ci skip] Removed explicit multi-threaded server dependency reference [ci skip] Revised Configuration and fixed typos [ci skip] Switched Lifecycle Graphic for Detailed Description [ci skip] Switched from Lifecycle graphic to detailed description to facilitate revision and maintainability in the future. Capitalized Heading (typo) [ci skip] Implemented merged commits from README [ci skip] Pulled over all (or at least I believe all) merged commits from README. (Dec 14, 2015 - Feb 11, 2016) Editorial - Capitalize WebSockets [ci skip] Reformated lines to ~75 characters [ci skip]
* Change `alpha` to `beta1` to prep for release of Rails 5eileencodes2015-12-1812-43/+43
| | | | :tada: :beers:
* Add task to test the release preparationRafael Mendonça França2015-12-182-2/+5
|
* Merge pull request #22668 from ryohashimoto/151219_receiveKasper Timm Hansen2015-12-181-1/+1
|\ | | | | Fix `receive` spelling
| * Fix `receive` spellingRyo Hashimoto2015-12-191-1/+1
| |
* | Merge pull request #22658 from greysteil/handle-specified-schema-in-index-removeMatthew Draper2015-12-192-3/+30
|\ \ | | | | | | Handle specified schemas when removing a Postgres index
| * | Handle specified schemas when removing a Postgres indexGrey Baker2015-12-182-3/+30
| | |
* | | Merge pull request #22667 from akshay-vishnoi/base_test_caseRafael França2015-12-181-0/+22
|\ \ \ | |_|/ |/| | [ActionCable] Test invalid action on channel
| * | [ActionCable] Test invalid action on channelAkshay Vishnoi2015-12-181-0/+22
| | |
* | | Fix `receive` spelling and add whitespaceKasper Timm Hansen2015-12-181-3/+3
|/ / | | | | | | | | Found `recieve` next to the correctly spelled method name, fixed it. Also we prefer a one space padding within hashes, add that.
* | Merge pull request #22666 from akshay-vishnoi/base_test_caseRafael França2015-12-181-1/+11
|\ \ | | | | | | [ActionCable] test perform action with default action
| * | [ActionCable] test perform action with default actionAkshay Vishnoi2015-12-181-1/+11
| | |
* | | Fix test that fails in isolationeileencodes2015-12-181-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This test needs to have a require for the Listen gem or else it returns an error when run by itself or with a certain order in the Active Support tests. We use `silence_warnings` because Listen has some warnings about private methods. It's already silenced when it's required in Active Support EventedFileUpdateChecker. ``` 1) Error: EventedFileUpdateCheckerTest#test_should_not_execute_the_block_if_no_paths_are_given: NameError: uninitialized constant EventedFileUpdateCheckerTest::Listen test/evented_file_update_checker_test.rb:21:in `teardown' ```
* | | Delete duplicate testeileencodes2015-12-181-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test was was in here twice so we were seeing this error: ``` activesupport/test/caching_test.rb:658: warning: method redefined; discarding old test_local_cache_of_read_nil activesupport/test/caching_test.rb:634: warning: previous definition of test_local_cache_of_read_nil was here ```
* | | Don't need to use `Array.wrap`Rafael Mendonça França2015-12-182-2/+2
|/ /
* | Merge pull request #22650 from derekprior/dp-actionpack-warningsRafael França2015-12-181-0/+5
|\ \ | | | | | | Fix "instance variable not initialized" in tests
| * | Fix "instance variable not initialized" in testsDerek Prior2015-12-171-0/+5
| | | | | | | | | | | | | | | | | | | | | The ActionPack test suite had a handful of these warnings when run. This was due to `assert_response` being tested outside the context of a controller instance where those instance variables would already have been initialized.
* | | Merge pull request #20797 from byroot/prevent-url-for-ac-parametersRafael França2015-12-186-14/+14
|\ \ \ | | | | | | | | Prevent ActionController::Parameters in url_for
| * | | Prevent ActionController::Parameters from being passed to url_for directlyJean Boussier2015-12-156-14/+14
| | | |
* | | | Merge pull request #22660 from y-yagi/add_line_break_between_methodsRafael França2015-12-181-1/+1
|\ \ \ \ | | | | | | | | | | add line break between method of generated channel js
| * | | | add line break between method of generated channel jsyuuji.yaginuma2015-12-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ``` # before App.appearance = App.cable.subscriptions.create "AppearanceChannel", connected: -> # Called when the subscription is ready for use on the server disconnected: -> # Called when the subscription has been terminated by the server received: (data) -> # Called when there's incoming data on the websocket for this channel appear: -> @perform 'appear' away: -> @perform 'away' ``` ``` # after App.appearance = App.cable.subscriptions.create "AppearanceChannel", connected: -> # Called when the subscription is ready for use on the server disconnected: -> # Called when the subscription has been terminated by the server received: (data) -> # Called when there's incoming data on the websocket for this channel appear: -> @perform 'appear' away: -> @perform 'away' ```
* | | | | Merge pull request #22665 from akshay-vishnoi/base_test_caseRafael França2015-12-181-0/+4
|\ \ \ \ \ | | | | | | | | | | | | [ActionCable] Test available actions on Channel
| * | | | | [ActionCable] Test available actions on ChannelAkshay Vishnoi2015-12-181-0/+4
| | | | | |
* | | | | | Do not check the version anymoreRafael Mendonça França2015-12-181-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Our logic is complex now and we don't need to check the version to asset the behavior so I'm removing the checking here.
* | | | | | Merge pull request #22657 from matthewd/loosen-versionRafael Mendonça França2015-12-184-3/+32
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | Allow normal version updates within a release series
| * | | | | Allow normal version updates within a release seriesMatthew Draper2015-12-184-3/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We originally chose to apply very strict versioning on the `rails` entry in the Gemfile, because our future versioning policy was not strongly defined. Now it is, and our policy is very much designed on the expectation that people will regularly update to the latest patch level in their release series... so we should encourage that. Of course, Gemfile.lock will do its job and prevent unplanned updates, just as it does for every other gem in the bundle... but if you run `bundle update`, we want to get you the latest bug/security fixes without requiring a manual edit of the Gemfile entry. Our current version could be a few different shapes, so it takes a bit of work to find the right specifier, but in principle, we match anything of the form x.y.*, where x.y matches our current release series.
* | | | | | Same gemspec formats everywhereDavid Heinemeier Hansson2015-12-182-2/+2
| | | | | |
* | | | | | Stray lineDavid Heinemeier Hansson2015-12-181-1/+2
| | | | | |
* | | | | | SimplifyDavid Heinemeier Hansson2015-12-181-9/+2
| | | | | |
* | | | | | Now available in action_cableDavid Heinemeier Hansson2015-12-181-2/+2
| | | | | |
* | | | | | Refer to rails command instead of rake in a bunch of placesDavid Heinemeier Hansson2015-12-1816-40/+40
| | | | | | | | | | | | | | | | | | | | | | | | Still more to do. Please assist!
* | | | | | Explain reason for eager loadingDavid Heinemeier Hansson2015-12-181-0/+1
| | | | | |
* | | | | | Explain Redis' role in Action CableDavid Heinemeier Hansson2015-12-181-0/+1
| | | | | |
* | | | | | [ci skip] Revert most of ff851017Kasper Timm Hansen2015-12-181-2/+2
| | | | | | | | | | | | | | | | | | We went back to `Thread.current[]` in 33e11e59.
* | | | | | Merge pull request #22656 from akshay-vishnoi/fix-typosKasper Timm Hansen2015-12-183-5/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | [ci skip] Fix grammar
| * | | | | | [ci skip] Fix grammarAkshay Vishnoi2015-12-183-5/+5
| | | | | | |
* | | | | | | Bring comment in line with rest of initializersDavid Heinemeier Hansson2015-12-181-2/+2
| | | | | | |
* | | | | | | Explain the concept of config initializers for backwards compatibilityDavid Heinemeier Hansson2015-12-183-2/+5
| | | | | | |
* | | | | | | Fix spacingDavid Heinemeier Hansson2015-12-181-1/+1
| | | | | | |
* | | | | | | These options are not changed nearly often enough to warrant inclusion like ↵David Heinemeier Hansson2015-12-181-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | this at a high level
* | | | | | | Better explanationDavid Heinemeier Hansson2015-12-181-2/+6
| | | | | | |
* | | | | | | Use the rails command, not rake, for all rails commandsDavid Heinemeier Hansson2015-12-182-6/+6
| | | | | | |
* | | | | | | README.md in the app is now used to describe the app itself, not RailsDavid Heinemeier Hansson2015-12-181-1/+0
| | | | | | |
* | | | | | | More sensible ordering with the more important options firstDavid Heinemeier Hansson2015-12-181-12/+12
| | | | | | |
* | | | | | | List the major frameworks you can remove togetherDavid Heinemeier Hansson2015-12-181-3/+3
| | | | | | |
* | | | | | | Revert "Use Thread.current.thread_variable_set/get insetad of the direct ↵David Heinemeier Hansson2015-12-181-4/+4
| |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | accessors" This reverts commit 301f43820562c6a70dffe30f4227ff0751f47d4f per @matthewd on https://github.com/rails/rails/pull/22630/files#r47997074
generated by cgit v1.2.3 (git 2.25.1) at 2024-08-31 20:48:00 +0000