Remove TODO comments

Author: mayorova

app/controllers/admin/api/buyers_users_controller.rb

@@ -131,8 +131,6 @@ def find_user
   def user_params
     @user_params ||= begin
                        allowed_attrs = user.defined_fields_names | %i(password password_confirmation)
-                       # TODO: are these parameters needed?
-                       # allowed_attrs |= %i(conditions cas_identifier open_id service_conditions)
                        flat_params.permit(*allowed_attrs)
                      end
   end

app/controllers/admin/api/users_controller.rb

@@ -141,8 +141,6 @@ def user_params
     @user_params ||= begin
                        allowed_attrs = user.defined_fields_names | %i(password password_confirmation cas_identifier)
                        allowed_attrs |= [member_permission_service_ids: [], member_permission_ids: [], allowed_sections: [], allowed_service_ids: []] if (provider_key.present? || current_user.admin?)
-                       # TODO: are these parameters needed?
-                       # allowed_attrs |= %i(conditions open_id service_conditions)
                        flat_params.permit(*allowed_attrs)
                      end
   end

app/controllers/buyers/users_controller.rb

@@ -87,8 +87,6 @@ def user_params
   def permitted_user_params
     @permitted_user_params ||= begin
                                  allowed_attrs = user.defined_builtin_fields_names | %i(password password_confirmation)
-                                 # TODO: are these parameters needed?
-                                 # allowed_attrs |= %i(conditions cas_identifier open_id service_conditions)
                                  user_params.permit(*allowed_attrs, extra_fields: user.defined_extra_fields_names)
                                end
   end

app/controllers/provider/admin/user/personal_details_controller.rb

@@ -32,8 +32,6 @@ def user_params
   def permitted_user_params
     @permitted_user_params ||= begin
                                  allowed_attrs = current_user.defined_builtin_fields_names | %i(password)
-                                 # TODO: are these parameters needed?
-                                 # allowed_attrs |= %i(conditions cas_identifier open_id service_conditions)
                                  user_params.permit(*allowed_attrs, extra_fields: current_user.defined_extra_fields_names)
                                end
   end

lib/developer_portal/app/controllers/developer_portal/accounts/invitee_signups_controller.rb

@@ -33,8 +33,7 @@ def sso_create
       redirect_to strategy.redirect_to_on_successful_login
     else
       user_data = strategy.user_data || {}
-      # TODO: verify that this is the right thing to do
-      user_attributes = user_data.to_hash.compact.slice(:username, :email)
+      user_attributes = user_data.to_hash.compact.slice(:username, :email, :authentication_id)
       @user.assign_attributes(user_attributes)
       session[:invitation_sso_uid] = user_data[:uid]
       session[:invitation_sso_system_name] = strategy.authentication_provider.system_name