Skip to content

Commit

Permalink
changelog: Upcoming Features, Select email workflow, select email sup…
Browse files Browse the repository at this point in the history
…ports single email (#11656)
  • Loading branch information
mdiarra3 authored Dec 18, 2024
1 parent db380f1 commit 1ea1bf5
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 15 deletions.
5 changes: 0 additions & 5 deletions app/controllers/sign_up/select_email_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ class SelectEmailController < ApplicationController
check_or_render_not_found -> { IdentityConfig.store.feature_select_email_to_share_enabled }
before_action :confirm_two_factor_authenticated
before_action :verify_needs_completions_screen
before_action :verify_multiple_emails

def show
@sp_name = current_sp.friendly_name || sp.agency&.name
Expand Down Expand Up @@ -56,10 +55,6 @@ def last_email
end
end

def verify_multiple_emails
redirect_to sign_up_completed_path if user_emails.count < 2
end

def verify_needs_completions_screen
redirect_to account_url unless needs_completion_screen_reason
end
Expand Down
10 changes: 0 additions & 10 deletions spec/controllers/sign_up/select_email_controller_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -61,16 +61,6 @@
end
end

context 'with only one verified email address' do
let(:user) { create(:user) }

it 'redirects to the sign up completed path' do
response

expect(response).to redirect_to(sign_up_completed_path)
end
end

context 'when users has max number of emails' do
before do
allow(user).to receive(:email_address_count).and_return(2)
Expand Down

0 comments on commit 1ea1bf5

Please sign in to comment.