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

Add admin buttons to delete and resubmit samples #45

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

lkeegan
Copy link
Member

@lkeegan lkeegan commented Oct 25, 2024

  • add DELETE admin/samples/<sample_id> API endpoint
    • deletes the sample and all associated input files and results
  • add delete button to admin interface with modal confirmation dialog
  • add POST admin/resubmit-sample/<sample_id> API endpoint
    • deletes any existing results for the sample, then sets its status to QUEUED
  • add resubmit button to admin interface with modal confirmation dialog

- add DELETE `admin/samples/<sample_id>` API endpoint
  - deletes the sample and all associated input files and results
- add delete button to admin interface with modal confirmation dialog
  - resolves #39
- add POST `admin/resubmit-sample/<sample_id>` API endpoint
  - deletes any existing results for the sample, then sets its status to QUEUED
- add resubmit button to admin interface with modal confirmation dialog
  - resolves #40
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 84.84848% with 5 lines in your changes missing coverage. Please review.

Project coverage is 78.82%. Comparing base (4fb00da) to head (d8b9477).

Files with missing lines Patch % Lines
backend/src/predicTCR_server/app.py 82.75% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
+ Coverage   78.18%   78.82%   +0.63%     
==========================================
  Files           8        8              
  Lines         651      680      +29     
==========================================
+ Hits          509      536      +27     
- Misses        142      144       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lkeegan lkeegan merged commit 4838031 into main Oct 25, 2024
5 checks passed
@lkeegan lkeegan deleted the fix_39_40_sample_delete_resubmit branch October 25, 2024 09:51
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 this pull request may close these issues.

Admin resubmit sample Admin delete samples
2 participants