aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Add a missing test case for the persistence behavior of `serialize`Sean Griffin2015-06-111-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 commitSean Griffin2015-06-111-1/+1
| | | | [Sean Griffin & jmondo]
* Correctly handle array columns with defaults in the schema dumperSean Griffin2015-06-114-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-apiSantiago Pastorino2015-06-1145-32/+1497
|\ | | | | Rails api
| * head :no_content is implicitly calledSantiago Pastorino2015-06-111-2/+0
| |
| * Mention that doing nothing in Rails API controllers returns 204Santiago Pastorino2015-06-111-1/+1
| |
| * Return 204 if render is not called in API controllersJorge Bejar2015-06-115-6/+31
| |
| * Add test coverage for implicit render in empty actionsJorge Bejar2015-06-111-0/+14
| |
| * Checking if controller responds to wrap_parameter is not longer requiredJorge Bejar2015-06-111-1/+1
| |
| * Enable wrap_parameter by default in rails api applicationsJorge Bejar2015-06-112-5/+1
| |
| * Include ParamsWrapper in AC::APIJorge Bejar2015-06-112-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 appsJorge Bejar2015-06-113-0/+20
| |
| * Make Rails API apps return the full resource on updateSantiago Pastorino2015-06-112-2/+2
| |
| * It's rails new my_api --apiSantiago Pastorino2015-06-111-1/+1
| |
| * Change guide heading from - to =Santiago Pastorino2015-06-111-1/+1
| |
| * Add CHANGELOG entries for API apps functionalitySantiago Pastorino2015-06-112-0/+13
| |
| * Add API only apps guideSantiago Pastorino2015-06-111-0/+435
| |
| * Document Generators.api_only! methodSantiago Pastorino2015-06-111-0/+4
| |
| * http only => API onlySantiago Pastorino2015-06-111-1/+1
| |
| * Add AMS 0.10.0.rc1 by default for api appsSantiago Pastorino2015-06-112-0/+6
| |
| * Fix scaffold generator test for resource routesSantiago Pastorino2015-06-111-1/+1
| |
| * Fix class_option description for api generatorsSantiago Pastorino2015-06-112-2/+2
| |
| * Remove unneeded option from ResourceRouteGeneratorSantiago Pastorino2015-06-111-3/+0
| |
| * Fix MimeResponds example in AC::API documentationSantiago Pastorino2015-06-111-3/+5
| |
| * Revert changes related with api apps in RouteWrapperJorge Bejar2015-06-111-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 testsJorge Bejar2015-06-111-2/+2
| |
| * Do not say that Api Controllers are faster than regular ones in docsJorge Bejar2015-06-111-4/+4
| |
| * Api only apps should include tmp and vendor foldersJorge Bejar2015-06-112-2/+6
| |
| * Remove Compatibility module since we don't remember why it was added :smile:Jorge Bejar2015-06-112-20/+0
| |
| * Exclude cache_digests:dependencies rake task in api appJorge Bejar2015-06-111-2/+4
| |
| * Routes resources avoid :new and :edit endpoints if api_only is enabledJorge Bejar2015-06-115-21/+111
| |
| * Remove extra whitespacesSantiago Pastorino2015-06-111-1/+1
| |
| * Use new hash syntaxSantiago Pastorino2015-06-113-8/+8
| |
| * Refactor internal? to query internal_controller? and internal_asset? methodsSantiago Pastorino2015-06-111-1/+11
| |
| * Remove Unneeded ApiPublicExceptions middleware, PublicExceptions already ↵Santiago Pastorino2015-06-113-84/+0
| | | | | | | | does the work
| * Remove api_rendering is not neededSantiago Pastorino2015-06-113-19/+1
| |
| * Adhere to Rails convention for private indentationSantiago Pastorino2015-06-111-21/+20
| |
| * Use nex hash syntax on testsSantiago Pastorino2015-06-111-5/+5
| |
| * Disable jbuilder for Rails API apps, meanwhile it doesn't play nicelySantiago Pastorino2015-06-112-0/+3
| |
| * Add test to show api only apps allow overriding generator optionsSantiago Pastorino2015-06-111-0/+16
| |
| * Add config.api_only = true to config/application.rb when using rails new --apiSantiago Pastorino2015-06-112-0/+11
| |
| * config.api_only = true implies config.generators.api_only = trueSantiago Pastorino2015-06-113-4/+8
| |
| * Do not generate lib/assets directory for api appsSantiago Pastorino2015-06-112-1/+2
| |
| * Api apps scaffold does not generate assetsSantiago Pastorino2015-06-113-1/+8
| |
| * Api apps scaffold does not generate helpersSantiago Pastorino2015-06-113-1/+6
| |
| * Api apps scaffold does not generate viewsSantiago Pastorino2015-06-113-2/+11
| |
| * Add api scaffold test for route, controller and its testsSantiago Pastorino2015-06-111-0/+55
| |
| * API apps scaffold generator generates an apropriate controllerSantiago Pastorino2015-06-116-2/+180
| |
| * config.generators.api_only = true set rails api option on generatorsSantiago Pastorino2015-06-112-1/+6
| |
| * Api apps scaffold generates routes without new and edit actionsSantiago Pastorino2015-06-112-1/+11
| |