diff --git a/app/controllers/aspects_controller.rb b/app/controllers/aspects_controller.rb index f58999e5b..d755a7268 100644 --- a/app/controllers/aspects_controller.rb +++ b/app/controllers/aspects_controller.rb @@ -19,7 +19,7 @@ class AspectsController < ApplicationController @selected_contacts.uniq! # redirect to signup - if (current_user.getting_started == true || @aspects.blank?) && !request.format.mobile? + if (current_user.getting_started == true || @aspects.blank?) && !request.format.mobile? && !request.format.js? redirect_to getting_started_path else if params[:sort_order].blank? and session[:sort_order].blank? diff --git a/spec/controllers/aspects_controller_spec.rb b/spec/controllers/aspects_controller_spec.rb index aa708001d..e1b421951 100644 --- a/spec/controllers/aspects_controller_spec.rb +++ b/spec/controllers/aspects_controller_spec.rb @@ -111,8 +111,25 @@ describe AspectsController do assigns(:posts).should == @posts.reverse end end + context 'with getting_started = true' do + before do + @alice.getting_started = true + @alice.save + end + it 'redirects to getting_started' do + get :index + response.should redirect_to getting_started_path + end + it 'does not redirect mobile users to getting_started' do + get :index, :format => :mobile + response.should_not be_redirect + end + it 'does not redirect ajax to getting_started' do + get :index, :format => :js + response.should_not be_redirect + end + end end - context 'performance', :performance => true do before do require 'benchmark'