[200] Build Manage Evaluators List

app/assets/stylesheets/application.sass.scss

@@ -9,3 +9,11 @@ dialog::backdrop {
   background-color: black;
   border-color: black;
 }
+
+.usa-table.usa-table--borderless.gray-header {
+  thead {
+    th {
+      background-color: #dfe1e2 !important;
+    }
+  }
+}

app/controllers/evaluator_invitations_controller.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+class EvaluatorInvitationsController < ApplicationController
+  before_action :set_challenge
+  before_action :set_evaluator_invitation
+
+  def resend_invitation
+    if @evaluator_invitation.update(last_invite_sent: Time.current)
+      # TODO: Implement sending the actual invitation email here
+      redirect_to challenge_manage_evaluators_path(@challenge),
+                  notice: t('.success')
+    else
+      redirect_to challenge_manage_evaluators_path(@challenge),
+                  alert: t('.failure')
+    end
+  end
+
+  private
+
+  def set_challenge
+    @challenge = Challenge.find(params[:challenge_id])
+  end
+
+  def set_evaluator_invitation
+    @evaluator_invitation = @challenge.evaluator_invitations.find(params[:id])
+  end
+end

app/controllers/manage_evaluators_controller.rb

@@ -0,0 +1,179 @@
+# frozen_string_literal: true
+
+class ManageEvaluatorsController < ApplicationController
+  include ManageEvaluatorsHelper
+
+  before_action :set_challenge
+
+  VALID_EVALUATOR_ROLES = %w[evaluator solver challenge_manager].freeze
+
+  def index
+    @phases = @challenge.phases.order(:start_date)
+
+    if @phases.empty?
+      handle_empty_phases
+    else
+      handle_existing_phases
+    end
+  end
+
+  def create
+    @phase = @challenge.phases.find(evaluator_invitation_params[:phase_id])
+    result = process_evaluator_invitation(evaluator_invitation_params[:email])
+
+    if result[:success]
+      handle_successful_creation(result)
+    else
+      handle_failed_creation
+    end
+  end
+
+  def destroy
+    @phase = @challenge.phases.find(params[:phase_id])
+    result = process_evaluator_removal(params[:evaluator_type], params[:evaluator_id])
+
+    render_json_response(result)
+  end
+
+  private
+
+  # Setup methods
+  def set_challenge
+    @challenge = Challenge.find(params[:challenge_id])
+  end
+
+  def evaluator_invitation_params
+    params.require(:evaluator_invitation).permit(
+      :first_name, :last_name, :email, :challenge_id, :phase_id, :last_invite_sent
+    )
+  end
+
+  # Index action helpers
+  def handle_empty_phases
+    flash.now[:alert] = t('.no_phases_alert')
+    @evaluator_invitations = []
+    @existing_evaluators = []
+  end
+
+  def handle_existing_phases
+    @phase = select_phase
+    @evaluator_invitations = fetch_evaluator_invitations
+    @existing_evaluators = fetch_existing_evaluators
+  end
+
+  def select_phase
+    params[:phase_id] ? @phases.find(params[:phase_id]) : @phases.first
+  end
+
+  def fetch_evaluator_invitations
+    @challenge.evaluator_invitations.where(phase: @phase)
+  end
+
+  def fetch_existing_evaluators
+    @challenge.evaluators.
+      joins(:challenge_phases_evaluators).
+      where(challenge_phases_evaluators: { phase: @phase }).
+      distinct
+  end
+
+  # Create action helpers
+  def process_evaluator_invitation(email)
+    existing_invitation = @challenge.evaluator_invitations.find_by(email:, phase: @phase)
+    user = User.find_by(email:)
+
+    if existing_invitation
+      resend_invitation(existing_invitation)
+    elsif user && valid_evaluator_role?(user)
+      add_user_as_evaluator(user)
+    else
+      create_new_invitation(email)
+    end
+  end
+
+  # prevent duplicate evaluator invitations
+  def resend_invitation(invitation)
+    invitation.update(last_invite_sent: Time.current) # only update last_invite_sent for now
+    {
+      success: true,
+      message: "An invitation to this challenge has already been sent to " \
+               "#{invitation.email}. Invitation has been resent."
+    }
+  end
+
+  def valid_evaluator_role?(user)
+    VALID_EVALUATOR_ROLES.include?(user.role)
+  end
+
+  def add_user_as_evaluator(user)
+    cpe = ChallengePhasesEvaluator.find_or_create_by(challenge: @challenge, phase: @phase, user:)
+    if cpe.persisted?
+      { success: true, message: "#{user.email} has been added as an evaluator for this phase." }
+    else
+      { success: false, message: "Failed to add #{user.email} as an evaluator." }
+    end
+  end
+
+  def create_new_invitation(email)
+    invitation = @challenge.evaluator_invitations.new(evaluator_invitation_params.merge(phase: @phase))
+    if invitation.save
+      { success: true, message: "Invitation sent to #{email} for this challenge phase." }
+    else
+      { success: false, message: invitation.errors.full_messages.join(", ") }
+    end
+  end
+
+  def handle_successful_creation(result)
+    redirect_to challenge_manage_evaluators_path(@challenge, phase_id: @phase.id),
+                notice: result[:message]
+  end
+
+  def handle_failed_creation
+    @evaluator_invitations = fetch_evaluator_invitations
+    @existing_evaluators = fetch_existing_evaluators
+    render :index
+  end
+
+  # Destroy action helpers
+  def process_evaluator_removal(evaluator_type, evaluator_id)
+    case evaluator_type
+    when 'user'
+      remove_user_evaluator(evaluator_id)
+    when 'invitation'
+      remove_evaluator_invitation(evaluator_id)
+    else
+      { success: false, message: 'Invalid evaluator type' }
+    end
+  end
+
+  def remove_user_evaluator(evaluator_id)
+    evaluator = @challenge.evaluators.find_by(id: evaluator_id)
+    return { success: false, message: 'Evaluator not found' } unless evaluator
+
+    cpe = ChallengePhasesEvaluator.find_by(challenge: @challenge, phase: @phase, user: evaluator)
+    if cpe&.destroy
+      { success: true, message: t('manage_evaluators.remove_user_evaluator.success') }
+    else
+      { success: false, message: t('manage_evaluators.remove_user_evaluator.failure') }
+    end
+  rescue StandardError => e
+    { success: false, message: "Error: #{e.message}" }
+  end
+
+  def remove_evaluator_invitation(invitation_id)
+    invitation = @challenge.evaluator_invitations.find_by(id: invitation_id, phase: @phase)
+    if invitation&.destroy
+      { success: true, message: t('manage_evaluators.remove_evaluator_invitation.success') }
+    else
+      { success: false, message: t('manage_evaluators.remove_evaluator_invitation.failure') }
+    end
+  end
+
+  def render_json_response(result)
+    if result[:success]
+      flash[:notice] = result[:message]
+      render json: { success: true, message: result[:message] }
+    else
+      render json: { success: false, message: result[:message] }, status: :unprocessable_entity
+    end
+  end
+end

app/helpers/manage_evaluators_helper.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+module ManageEvaluatorsHelper
+  def user_status(evaluator, challenge)
+    if evaluator.is_a?(User)
+      user_status_for_existing_user(evaluator, challenge)
+    else
+      "Invite Sent"
+    end
+  end
+
+  def assigned_submissions_count(evaluator, challenge, phase)
+    if evaluator.is_a?(User)
+      evaluator.evaluator_submission_assignments.
+        joins(:submission).
+        where(submissions: { challenge:, phase: }).
+        count
+    else
+      0
+    end
+  end
+
+  private
+
+  def user_status_for_existing_user(user, challenge)
+    if challenge.challenge_phases_evaluators.exists?(user_id: user.id)
+      user.status == 'active' ? "Available" : "Awaiting Approval"
+    else
+      "Invite Sent"
+    end
+  end
+end

app/javascript/application.js

@@ -1 +1,2 @@
 import "./controllers"
+import "./delete_evaluator_modal"

app/javascript/delete_evaluator_modal.js

@@ -0,0 +1,77 @@
+export function initializeDeleteEvaluatorModal() {
+  let currentEvaluatorId, currentEvaluatorType, challengeId, phaseId;
+
+  document.querySelectorAll('[data-open-modal]').forEach(button => {
+    button.addEventListener('click', (e) => {
+      e.preventDefault();
+      e.stopPropagation();
+      currentEvaluatorId = button.dataset.evaluatorId;
+      currentEvaluatorType = button.dataset.evaluatorType;
+      challengeId = button.dataset.challengeId;
+      phaseId = button.dataset.phaseId;
+      const modal = document.getElementById('delete-evaluator-modal');
+      modal.classList.add('is-visible');
+      return false;
+    });
+  });
+
+  document.getElementById('confirmDelete').addEventListener('click', () => {
+    deleteEvaluator();
+  });
+
+  function deleteEvaluator(forceDelete = false) {
+    const csrfToken = document.querySelector('meta[name="csrf-token"]').content;
+    const challengeId = document.querySelector('[data-challenge-id]').dataset.challengeId;
+
+    fetch(`/challenges/${challengeId}/manage_evaluators`, {
+      method: 'DELETE',
+      headers: {
+        'Content-Type': 'application/json',
+        'X-CSRF-Token': csrfToken
+      },
+      body: JSON.stringify({
+        evaluator_id: currentEvaluatorId,
+        evaluator_type: currentEvaluatorType,
+        phase_id: phaseId,
+        force_delete: forceDelete
+      })
+    })
+    .then(response => {
+      if (!response.ok) {
+        throw new Error('Network response was not ok');
+      }
+      return response.json();
+    })
+    .then(data => {
+      if (data.success) {
+        const modal = document.getElementById('delete-evaluator-modal');
+        modal.classList.remove('is-visible');
+        window.location.reload();
+      } else {
+        throw new Error(data.message || 'Failed to remove evaluator');
+      }
+    })
+    .catch(error => {
+      console.error('Error:', error);
+      alert(error.message || 'An error occurred while removing the evaluator');
+    });
+  }
+
+  document.querySelectorAll('[data-close-modal]').forEach(element => {
+    element.addEventListener('click', () => {
+      const modal = document.getElementById('delete-evaluator-modal');
+      modal.classList.remove('is-visible');
+
+      const modalDescription = document.getElementById('modal-1-description');
+      modalDescription.textContent = 'Deleting an evaluator from the challenge will remove the evaluator from any submissions of this challenge that the evaluator is assigned to. It will also delete any of their completed or in progress evaluations for those submissions.';
+
+      const confirmButton = document.getElementById('confirmDelete');
+      confirmButton.textContent = 'Yes';
+      confirmButton.onclick = () => deleteEvaluator();
+    });
+  });
+}
+
+if (document.querySelector('[data-page="manage-evaluators"]')) {
+  document.addEventListener('DOMContentLoaded', initializeDeleteEvaluatorModal);
+}

app/views/manage_evaluators/_delete_evaluator_modal.html.erb

@@ -0,0 +1,46 @@
+<div
+  class="usa-modal"
+  id="delete-evaluator-modal"
+  aria-labelledby="modal-1-heading"
+  aria-describedby="modal-1-description"
+>
+  <div class="usa-modal__content">
+    <div class="usa-modal__main">
+      <h2 class="usa-modal__heading" id="modal-1-heading">
+        Are you sure you want to delete an evaluator from this challenge?
+      </h2>
+      <div class="usa-prose">
+        <p id="modal-1-description">
+          Deleting an evaluator from the challenge will remove the evaluator from any submissions of this challenge that the evaluator is assigned to. It will also delete any of their completed or in progress evaluations for those submissions.
+        </p>
+      </div>
+      <div class="usa-modal__footer">
+        <ul class="usa-button-group">
+          <li class="usa-button-group__item">
+            <button type="button" class="usa-button" id="confirmDelete">
+              Yes
+            </button>
+          </li>
+          <li class="usa-button-group__item padding-top-1 text-center">
+            <a
+              href="#"
+              class="usa-link"
+              data-close-modal
+            >
+              Back
+            </a>
+          </li>
+        </ul>
+      </div>
+    </div>
+    <button
+      type="button"
+      class="usa-button usa-modal__close"
+      aria-label="Close this window"
+      data-close-modal
+    >
+      <svg class="usa-icon" aria-hidden="true" focusable="false" role="img">
+        <use xlink:href="/assets/img/sprite.svg#close"></use>
+      </svg>
+    </button>
+  </div>