Add assistants by email

Author: nygrenh

app/controllers/setup/course_assistants_controller.rb

@@ -21,7 +21,7 @@ def create
     authorize! :modify_assistants, @course
 
     if params[:commit] == 'Add new assistant'
-      new_assistant = User.find_by(login: assistant_params[:username])
+      new_assistant = User.find_by('lower(email) = ?', assistant_params[:email].downcase)
       @assistantship = Assistantship.new(user: new_assistant, course: @course)
 
       if @assistantship.save
@@ -54,6 +54,6 @@ def destroy
   private
 
     def assistant_params
-      params.permit(:username)
+      params.permit(:email)
     end
 end

app/views/setup/course_assistants/index.html.erb

@@ -20,8 +20,8 @@
     <%= render 'shared/error_messages', target: @assistantship %>
 
     <div class="field">
-      <%= label_tag :username, "Enter assistant's username:" %>
-      <%= text_field_tag :username, nil, class: 'form-control' %>
+      <%= label_tag :email, "Enter assistant's email:" %>
+      <%= text_field_tag :email, nil, class: 'form-control' %>
     </div>
     <br>
     <div class="actions">
@@ -50,7 +50,7 @@
     <tbody>
     <% @assistants.each do |assistant| %>
         <tr>
-          <td><%= link_to assistant.username, participant_path(assistant) %></td>
+          <td><%= link_to assistant.email, participant_path(assistant) %></td>
           <td><%= link_to 'Remove', setup_organization_course_course_assistant_path(@course.organization, @course, assistant.assistantship(@course)), method: :delete, class: "btn btn-sm btn-warning", data: { confirm: 'Are you sure?' } %></td>
         </tr>
     <% end %>