I thought we decided that index will handle show_many. Returning all items is just a special case of returning a subset of items.

-----Original Message-----
From: "Andrew White" <notifications@github.com>
Sent: ‎01-‎08-‎2013 01:54
To: "ujjwalt/rails" <rails@noreply.github.com>
Cc: "Ujjwal Thaakar" <ujjwalthaakar@gmail.com>
Subject: Re: [rails] Default Collection Routing (#1)

In actionpack/lib/action_dispatch/routing/mapper.rb:
> @@ -948,8 +948,8 @@ module Resources
>          # CANONICAL_ACTIONS holds all actions that does not need a prefix or
>          # a path appended since they fit properly in their scope level.
>          VALID_ON_OPTIONS  = [:new, :collection, :member]
> -        RESOURCE_OPTIONS  = [:as, :controller, :path, :only, :except, :param, :concerns]
> -        CANONICAL_ACTIONS = %w(index create new show update destroy)
> +        RESOURCE_OPTIONS  = [:as, :controller, :path, :only, :except, :param, :concerns, :bulk]
> +        CANONICAL_ACTIONS = %w(index create new show update destroy replace update_many destroy_many)
Shouldn't :show_many be in here as well?
—
Reply to this email directly or view it on GitHub.