Commit d7553d36 authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Address review comments

Signed-off-by: default avatarDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
parent 73e00301
...@@ -18,12 +18,10 @@ class Admin::IdentitiesController < Admin::ApplicationController ...@@ -18,12 +18,10 @@ class Admin::IdentitiesController < Admin::ApplicationController
end end
def destroy def destroy
respond_to do |format| if @identity.destroy
if @identity.destroy redirect_to admin_user_identities_path(@user), notice: 'User identity was successfully removed.'
format.html { redirect_to admin_user_identities_path(@user), notice: 'User identity was successfully removed.' } else
else redirect_to admin_user_identities_path(@user), alert: 'Failed to remove user identity.'
format.html { redirect_to admin_user_identities_path(@user), alert: 'Failed to remove user identity.' }
end
end end
end end
...@@ -38,6 +36,6 @@ class Admin::IdentitiesController < Admin::ApplicationController ...@@ -38,6 +36,6 @@ class Admin::IdentitiesController < Admin::ApplicationController
end end
def identity_params def identity_params
params[:identity].permit(:provider, :extern_uid) params[:identity].require(:identity).permit(:provider, :extern_uid)
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment