Skip to content

Commit cf56fbe

Browse files
author
Julián Berbel Alt
committed
Remove second confirmation step
1 parent fa10f2c commit cf56fbe

File tree

5 files changed

+5
-46
lines changed

5 files changed

+5
-46
lines changed

app/controllers/users_controller.rb

+5-11
Original file line numberDiff line numberDiff line change
@@ -47,25 +47,19 @@ def exam_authorizations
4747
@exam_authorization_requests ||= ExamAuthorizationRequest.where(user: current_user, organization: Organization.current)
4848
end
4949

50-
def delete_request
51-
end
52-
5350
def send_delete_confirmation_email
5451
current_user.generate_delete_account_token!
55-
UserMailer.delete_account(current_user).deliver_now
52+
UserMailer.delete_account(current_user).post!
5653
redirect_to delete_request_user_path
5754
end
5855

5956
def delete_confirmation
60-
redirect_to delete_confirmation_invalid_user_path unless @user.delete_account_token_matches? params[:token]
61-
end
57+
return redirect_to delete_confirmation_invalid_user_path unless @user.delete_account_token_matches? params[:token]
6258

63-
def delete_confirmation_invalid
64-
end
59+
@user.destroy!
6560

66-
def disable
67-
current_user.disable!
68-
redirect_to root_path
61+
flash.notice = I18n.t(:user_deleted_successfully)
62+
redirect_to logout_path
6963
end
7064

7165
def permissible_params

app/views/users/_user_delete_account.html.erb

-19
This file was deleted.

app/views/users/delete_confirmation.html.erb

-11
This file was deleted.
-4
Original file line numberDiff line numberDiff line change
@@ -1,5 +1 @@
11
<%= render partial: 'user_delete_confirmation' %>
2-
3-
<%= content_for :no_container do %>
4-
<%= render partial: 'user_delete_modal', locals: { submit_url: delete_request_user_url } %>
5-
<% end %>

config/routes.rb

-1
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,6 @@
7979
post :delete_request, to: 'users#send_delete_confirmation_email'
8080
get :delete_confirmation_invalid
8181
get :delete_confirmation
82-
post :delete_confirmation, to: 'users#disable'
8382
end
8483

8584
resources :faqs, only: [:index]

0 commit comments

Comments
 (0)