Totally correct
On 26/04/2014 10:42 am, "Robert Guthrie" <notifications@github.com> wrote:

> In app/controllers/groups/manage_membership_requests_controller.rb:
>
> > @@ -5,6 +5,10 @@ class Groups::ManageMembershipRequestsController < GroupBaseController
> >    def index
> >      @group = GroupDecorator.new Group.find(params[:group_id])
> >      if can? :manage_membership_requests, @group
> > +      @current_requests = @group.membership_requests.where('response IS ?', nil)
> > +      @previous_requests = @group.membership_requests.where('response IS NOT ?', nil).
>
> This should be a scope. How about 'pending', and 'responded_to'.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/loomio/loomio/pull/1463/files#r12019248>
> .
>