Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft submissions should not be visible to challenge managers #365

Open
stepchud opened this issue Jan 16, 2025 · 1 comment · Fixed by #364
Open

Draft submissions should not be visible to challenge managers #365

stepchud opened this issue Jan 16, 2025 · 1 comment · Fixed by #364
Assignees

Comments

@stepchud
Copy link
Contributor

stepchud commented Jan 16, 2025

On Phoenix, draft submissions from solvers are not available to admins or challenge managers. Submissions on Rails are not filtering out draft submissions and so the counter_cache column phases.submissions_count includes draft submissions in the count and also the phase submissions list displays them. Draft Submissions can be assigned to evaluators, etc.

How to reproduce

Create Submission

  1. As a Solver, create a draft submission for a challenge phase, via the normal web submission page.
  2. As a Challenge Manager, try to manage your "Submissions & Evaluations"

Expected behavior

  • Draft submissions should not be factored into submission counts and challenge managers should not be able to see them listed in their Phase Submissions view. Evaluators should not be able to be assigned to draft submissions.

Actual behavior

submissions_count includes drafts and the draft submissions are visible on the app.

Technical Notes

Phoenix routes

  • Web.SubmissionController.create/2 ("action" => "draft") ("action" => "review")
  • Web.SubmissionController.update/2 ("action" => "review" to submit)
  • Web.SubmissionController.delete
  • The route at Web.SubmissionController.submit/2 and view functions Web.SubmissionView.submit_button/4, Web.SubmissionView.submit_button2/4 seem to be dead code.
@stepchud
Copy link
Contributor Author

@r-bartlett-gsa this is ready to test, it is similar to #370 except it only applies to Draft Submission status.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants