aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Merge pull request #24405 from waits/shallow-falseRafael França2019-04-022-1/+35
|\ \ \ \ | | | | | | | | | | Honor shallow: false on nested resources
| * | | | Honor shallow: false on nested resourcesDylan Waits2016-04-032-1/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously there was no way to place a non-shallow resource inside a parent with `shallow: true` set. Now you can set `shallow: false` on a nested child resource to generate normal (non-shallow) routes for it. Fixes #23890.
* | | | | Merge pull request #35805 from excid3/attachments-field-generatorsRafael França2019-04-0213-25/+130
|\ \ \ \ \ | | | | | | | | | | | | Add attachment and attachments field generators
| * | | | | Add attachment and attachments field generatorsChris Oliver2019-03-3013-25/+130
| | | | | |
* | | | | | Merge pull request #35771 from timoschilling/hash-speed-improvementsRafael França2019-04-022-4/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | Hash / HashWithIndifferentAccess speed improvements
| * | | | | | Speed improvement for HashWithIndifferentAccess#values_atTimo Schilling2019-04-021-2/+2
| | | | | | |
| * | | | | | Speed improvement for HashWithIndifferentAccess#fetch_valuesTimo Schilling2019-04-021-1/+1
| | | | | | |
| * | | | | | Speed improvement for HashWithIndifferentAccess#exceptTimo Schilling2019-04-011-0/+3
| | | | | | |
| * | | | | | Speed improvement for Hash#exceptTimo Schilling2019-03-291-1/+1
| | | | | | |
* | | | | | | Simplify ActionCable.createWebSocketURL and realphabetize exports (#35810)rmacklin2019-04-024-12/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove unnecessary variable from ActionCable.createWebSocketURL * Improve ActionCable test by creating the Consumer before reassigning URL With this change, the test now actually verifies that the Consumer's url property changes dynamically (from testURL to `${testURL}foo`). * Fix alphabetization of ActionCable exports
* | | | | | | Merge pull request #35768 from robertomiranda/r/rake-db-prepareEileen M. Uchitelle2019-04-023-0/+46
|\ \ \ \ \ \ \ | |_|_|_|/ / / |/| | | | | | Add db:prepare rake task.
| * | | | | | Add migration in case of db:drop assertionRoberto Miranda2019-04-021-0/+7
| | | | | | |
| * | | | | | 💇Roberto Miranda2019-04-023-8/+5
| | | | | | |
| * | | | | | Add test cases for rake db:prepareRoberto Miranda2019-04-023-6/+39
| | | | | | |
| * | | | | | Move db:migrate call to else blockRoberto Miranda2019-03-271-1/+2
| | | | | | |
| * | | | | | Improve db:prepare descriptionRoberto Miranda2019-03-271-1/+1
| | | | | | |
| * | | | | | Add rake db:prepare rake task.Roberto Miranda2019-03-271-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It Creates the database, loads the schema, run the migrations and initializes with the seed data (use db:reset to also drop the database first). This rake task runs in an idempotent way ref https://github.com/rails/rails/pull/33139#discussion_r195930751
* | | | | | | Merge pull request #35831 from ↵Kasper Timm Hansen2019-04-021-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | abhaynikam/35826-updated-doc-template-rawfile-after-35826 [ci skip] Updated the doc after renaming Template::File -> Template::RawFile in #35826
| * | | | | | | [ci skip] Updated the doc after renaming Template::File -> Template::RawFile ↵Abhay Nikam2019-04-021-1/+1
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | in #35826
* | | | | | | Merge pull request #35826 from cpruitt/rename-file-to-rawfileKasper Timm Hansen2019-04-025-5/+5
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Rename `Template::File` to `Template::RawFile`
| * | | | | | | Rename File to RawFileCliff Pruitt2019-04-015-5/+5
| | |_|_|_|/ / | |/| | | | |
* | | | | | | Merge pull request #35703 from y-yagi/add_database_option_to_dbconsole_commandYuji Yaginuma2019-04-023-13/+42
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Rename `connection` option to `database` in `dbconsole` command
| * | | | | | | Rename `connection` option to `database` in `dbconsole` commandyuuji.yaginuma2019-04-023-13/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We introduced `connection` option for specifying spec with 1acd9a6464668d4d54ab30d016829f60b70dbbeb. But now we are using the `database` to specify the same value in other commands. * https://github.com/rails/rails/blob/0a0f115031b64b5335fa88543c40df4194dfb428/activerecord/lib/rails/generators/active_record/migration/migration_generator.rb#L11 * https://github.com/rails/rails/blob/0a0f115031b64b5335fa88543c40df4194dfb428/activerecord/lib/rails/generators/active_record/model/model_generator.rb#L17 The options provided to the users should be uniform. Since the term "database" is used in rake task etc, So I want to be able to use it in `dbconsole` command. Also I deprecated the `connection` option because I think that it would be confusing if there are multiple options to specify a same value.
* | | | | | | | Merge pull request #35827 from abhaynikam/35824-fix-typo-in-changelogRyuta Kamizono2019-04-021-2/+2
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | [ci skip] Fix typo for recieve -> receive
| * | | | | | | [ci skip] Fix typo for recieve -> receiveAbhay Nikam2019-04-021-2/+2
|/ / / / / / /
* | | | | | | Merge pull request #35824 from connorshea/add-changelog-for-webdriversRafael França2019-04-011-0/+6
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Add Changelog entry for #35732.
| * | | | | | Add Changelog entry for #35732.Connor Shea2019-04-011-0/+6
|/ / / / / /
* | | | | | [ci skip] Follow up c8bf334104Kasper Timm Hansen2019-04-011-1/+2
| | | | | |
* | | | | | Only clear template caches in dev after changes (#35629)John Hawthorn2019-04-016-7/+61
| | | | | |
* | | | | | Merge pull request #35820 from sharang-d/url-to-URL-except-actionpackRyuta Kamizono2019-04-0226-41/+41
|\ \ \ \ \ \ | | | | | | | | | | | | | | url -> URL where apt except inside actionpack/
| * | | | | | url -> URL where apt except inside actionpack/Sharang Dashputre2019-04-0126-41/+41
| | | | | | |
* | | | | | | Merge pull request #35821 from sharang-d/url-to-URL-in-actionpackRyuta Kamizono2019-04-0210-24/+24
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | url -> URL where apt inside actionpack/
| * | | | | | | url -> URL where apt inside actionpack/Sharang Dashputre2019-04-0110-24/+24
| |/ / / / / /
* | | | | | | Merge pull request #35819 from orhantoy/ruby-version-with-final-newlineRyuta Kamizono2019-04-022-1/+3
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Include final newline in generated .ruby-version
| * | | | | | Include final newline in generated .ruby-versionOrhan Toy2019-04-012-1/+3
|/ / / / / /
* | | | | | Merge pull request #35816 from ↵Ryuta Kamizono2019-04-0215-147/+11
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | kamipo/partly_revert_query_annotation_for_update_and_delete Revert unused code and re-using query annotation for `update_all` and `delete_all`
| * | | | | | Revert unused code and re-using query annotation for `update_all` and ↵Ryuta Kamizono2019-04-0115-147/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `delete_all` This partly reverts #35617. #35617 includes unused code (for `InsertStatement`) and re-using query annotation for `update_all` and `delete_all`, which has not been discussed yet. If a relation has any annotation, I think it is mostly for SELECT query, so re-using annotation by default is not always desired behavior for me. We should discuss about desired behavior before publishing the implementation.
* | | | | | | Merge pull request #35815 from sharang-d/update-action-cable-docsXavier Noria2019-04-011-3/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | url -> URL in Action Cable guide [ci skip]
| * | | | | | | url -> URL in Action Cable guide [ci skip]Sharang Dashputre2019-04-011-3/+3
| |/ / / / / /
* | | | | | | Merge pull request #35814 from saiqulhaq/masterMatthew Draper2019-04-011-2/+2
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Fix guides to link to edgeapi if on EDGE env
| * | | | | | [ci skip] Fix guides to link to edgeapi if on EDGE envM. Saiqul Haq2019-04-011-2/+2
|/ / / / / /
* | | | | | Fix typo in the value of ENCRYPTED_SIGNED_COOKIE_SALT constant (#35619)Emil Shakirov2019-04-012-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix typo in the value of ENCRYPTED_SIGNED_COOKIE_SALT constant * Fix value of cookie in with authenticated encryption flag off in tests [Emil Shakirov + Ryuta Kamizono]
* | | | | | Merge pull request #35727 from zinosama/zzz/update-doc-for-includesRyuta Kamizono2019-04-011-1/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update API doc for #includes on unnecessary #references [ci skip]
| * | | | | | [ci skip] Update doc for unnecessary #referenceszino2019-03-231-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Update doc for #includes to clarify that #references is unnecessary when conditions are passed into #includes as a hash.
* | | | | | | Revert "Add ActiveStorage.service_configurations and ActiveStorage.service"George Claghorn2019-03-314-36/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 002684e36e813469c3585e193f0698784c88278b.
* | | | | | | Extract insert test case from #35686Kasper Timm Hansen2019-03-311-0/+14
| | | | | | |
* | | | | | | Use accessors internally; remove needless validationKasper Timm Hansen2019-03-311-6/+2
| | | | | | |
* | | | | | | Mark InsertAll as private API. Easier to add later.Kasper Timm Hansen2019-03-311-1/+1
| | | | | | |
* | | | | | | Capture some join calls.Kasper Timm Hansen2019-03-311-4/+8
| | | | | | |
* | | | | | | feat(js): Dynamic ActionCable URL (#35579)Ryan Castner2019-03-315-32/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Failing test case * feat: Dynamic Url Generation Change createWebSocketURL to be a closure that allows url to be evaluated at the time the webSocket is established * refactor: createWebSocketURL to Consumer, remove need for closure Move initial call to createWebSocketURL in createConsumer * docs: Add documentation for dynamic url and string args to createConsumer Co-Authored-By: rmacklin <rmacklin@users.noreply.github.com> [Ryan Castner, rmacklin]