Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add a missing test case for the persistence behavior of `serialize` | Sean Griffin | 2015-06-11 | 1 | -0/+13 |
| | | | | | | | | | | | | | | | | | `serialize` makes the contract that if it is given a class name, it will never return something other than an instance of that class. This means that it must cast `nil` to the empty form of that object. As such, we should then persist empty forms of that object as `nil`. While this is techincally under the contract of ``` model.attribute = value assert_equal model.attribute, model.tap(&:save).reload.attribute ``` which we can't actually test universally without property based testing, it has come up more than once and is worth calling out specifically since we aren't looking to change it. | ||||
* | Credit the author of #20515 in the previous commit | Sean Griffin | 2015-06-11 | 1 | -1/+1 |
| | | | | [Sean Griffin & jmondo] | ||||
* | Correctly handle array columns with defaults in the schema dumper | Sean Griffin | 2015-06-11 | 4 | -0/+17 |
| | | | | | | | | | If the subtype provides custom schema dumping behavior, we need to defer to it. We purposely choose not to handle any values other than an array (which technically should only ever be `nil`, but I'd rather code defensively here). Fixes #20515. | ||||
* | Merge pull request #19832 from spastorino/rails-api | Santiago Pastorino | 2015-06-11 | 45 | -32/+1497 |
|\ | | | | | Rails api | ||||
| * | head :no_content is implicitly called | Santiago Pastorino | 2015-06-11 | 1 | -2/+0 |
| | | |||||
| * | Mention that doing nothing in Rails API controllers returns 204 | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 |
| | | |||||
| * | Return 204 if render is not called in API controllers | Jorge Bejar | 2015-06-11 | 5 | -6/+31 |
| | | |||||
| * | Add test coverage for implicit render in empty actions | Jorge Bejar | 2015-06-11 | 1 | -0/+14 |
| | | |||||
| * | Checking if controller responds to wrap_parameter is not longer required | Jorge Bejar | 2015-06-11 | 1 | -1/+1 |
| | | |||||
| * | Enable wrap_parameter by default in rails api applications | Jorge Bejar | 2015-06-11 | 2 | -5/+1 |
| | | |||||
| * | Include ParamsWrapper in AC::API | Jorge Bejar | 2015-06-11 | 2 | -1/+31 |
| | | | | | | | | | | | | | | | | | | ParamsWrapper was initially removed from API controllers according to the following discusision: https://github.com/rails-api/rails-api/issues/33 However, we're including it again so Rails API devs can decide whether to enable or disable it. | ||||
| * | Add rake-cors gem (commented) in Gemfile for rails api apps | Jorge Bejar | 2015-06-11 | 3 | -0/+20 |
| | | |||||
| * | Make Rails API apps return the full resource on update | Santiago Pastorino | 2015-06-11 | 2 | -2/+2 |
| | | |||||
| * | It's rails new my_api --api | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 |
| | | |||||
| * | Change guide heading from - to = | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 |
| | | |||||
| * | Add CHANGELOG entries for API apps functionality | Santiago Pastorino | 2015-06-11 | 2 | -0/+13 |
| | | |||||
| * | Add API only apps guide | Santiago Pastorino | 2015-06-11 | 1 | -0/+435 |
| | | |||||
| * | Document Generators.api_only! method | Santiago Pastorino | 2015-06-11 | 1 | -0/+4 |
| | | |||||
| * | http only => API only | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 |
| | | |||||
| * | Add AMS 0.10.0.rc1 by default for api apps | Santiago Pastorino | 2015-06-11 | 2 | -0/+6 |
| | | |||||
| * | Fix scaffold generator test for resource routes | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 |
| | | |||||
| * | Fix class_option description for api generators | Santiago Pastorino | 2015-06-11 | 2 | -2/+2 |
| | | |||||
| * | Remove unneeded option from ResourceRouteGenerator | Santiago Pastorino | 2015-06-11 | 1 | -3/+0 |
| | | |||||
| * | Fix MimeResponds example in AC::API documentation | Santiago Pastorino | 2015-06-11 | 1 | -3/+5 |
| | | |||||
| * | Revert changes related with api apps in RouteWrapper | Jorge Bejar | 2015-06-11 | 1 | -11/+1 |
| | | | | | | | | | | See the following commit to have context about this change: https://github.com/rails/rails/commit/757a2bc3e3e52a5d9418656928db993db42b741b | ||||
| * | Rename test methods in api conditional get controller tests | Jorge Bejar | 2015-06-11 | 1 | -2/+2 |
| | | |||||
| * | Do not say that Api Controllers are faster than regular ones in docs | Jorge Bejar | 2015-06-11 | 1 | -4/+4 |
| | | |||||
| * | Api only apps should include tmp and vendor folders | Jorge Bejar | 2015-06-11 | 2 | -2/+6 |
| | | |||||
| * | Remove Compatibility module since we don't remember why it was added :smile: | Jorge Bejar | 2015-06-11 | 2 | -20/+0 |
| | | |||||
| * | Exclude cache_digests:dependencies rake task in api app | Jorge Bejar | 2015-06-11 | 1 | -2/+4 |
| | | |||||
| * | Routes resources avoid :new and :edit endpoints if api_only is enabled | Jorge Bejar | 2015-06-11 | 5 | -21/+111 |
| | | |||||
| * | Remove extra whitespaces | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 |
| | | |||||
| * | Use new hash syntax | Santiago Pastorino | 2015-06-11 | 3 | -8/+8 |
| | | |||||
| * | Refactor internal? to query internal_controller? and internal_asset? methods | Santiago Pastorino | 2015-06-11 | 1 | -1/+11 |
| | | |||||
| * | Remove Unneeded ApiPublicExceptions middleware, PublicExceptions already ↵ | Santiago Pastorino | 2015-06-11 | 3 | -84/+0 |
| | | | | | | | | does the work | ||||
| * | Remove api_rendering is not needed | Santiago Pastorino | 2015-06-11 | 3 | -19/+1 |
| | | |||||
| * | Adhere to Rails convention for private indentation | Santiago Pastorino | 2015-06-11 | 1 | -21/+20 |
| | | |||||
| * | Use nex hash syntax on tests | Santiago Pastorino | 2015-06-11 | 1 | -5/+5 |
| | | |||||
| * | Disable jbuilder for Rails API apps, meanwhile it doesn't play nicely | Santiago Pastorino | 2015-06-11 | 2 | -0/+3 |
| | | |||||
| * | Add test to show api only apps allow overriding generator options | Santiago Pastorino | 2015-06-11 | 1 | -0/+16 |
| | | |||||
| * | Add config.api_only = true to config/application.rb when using rails new --api | Santiago Pastorino | 2015-06-11 | 2 | -0/+11 |
| | | |||||
| * | config.api_only = true implies config.generators.api_only = true | Santiago Pastorino | 2015-06-11 | 3 | -4/+8 |
| | | |||||
| * | Do not generate lib/assets directory for api apps | Santiago Pastorino | 2015-06-11 | 2 | -1/+2 |
| | | |||||
| * | Api apps scaffold does not generate assets | Santiago Pastorino | 2015-06-11 | 3 | -1/+8 |
| | | |||||
| * | Api apps scaffold does not generate helpers | Santiago Pastorino | 2015-06-11 | 3 | -1/+6 |
| | | |||||
| * | Api apps scaffold does not generate views | Santiago Pastorino | 2015-06-11 | 3 | -2/+11 |
| | | |||||
| * | Add api scaffold test for route, controller and its tests | Santiago Pastorino | 2015-06-11 | 1 | -0/+55 |
| | | |||||
| * | API apps scaffold generator generates an apropriate controller | Santiago Pastorino | 2015-06-11 | 6 | -2/+180 |
| | | |||||
| * | config.generators.api_only = true set rails api option on generators | Santiago Pastorino | 2015-06-11 | 2 | -1/+6 |
| | | |||||
| * | Api apps scaffold generates routes without new and edit actions | Santiago Pastorino | 2015-06-11 | 2 | -1/+11 |
| | |