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

CRDCDH-2187 Submission Request Cancel/Restore #602

Open
wants to merge 14 commits into
base: 3.2.0
Choose a base branch
from
Open

Conversation

amattu2
Copy link
Member

@amattu2 amattu2 commented Jan 22, 2025

Overview

This PR builds out the functionality to Cancel and Restore a Submission Request, and includes supporting QoL updates for related implementations.

Warning

Some implementation notes:

  • The restore functionality is implemented using assumptions, since the API isn't currently merged
  • The backend refers to the "cancel" action API as "delete", so in the frontend I remapped that to cancelApplication instead of deleteApplication since that's the correct terminology
  • When you delete an application, it disappears from the list. Once we have the filters in place, I assume it would be visible still (if you select the Canceled status)

Change Details (Specifics)

  • Add a Cancel/Restore button for the Submission Request List
  • Add associated test coverage
  • Update the GenericTable to mitigate 508 errors when using an empty column header name (Instead of using a TR, switch it to a TD)

Related Ticket(s)

CRDCDH-2187 (Task)
CRDCDH-2146 (US 1)
CRDCDH-2142 (US 2)

@amattu2 amattu2 added the 🚧 Do Not Merge This PR is not ready for merging label Jan 22, 2025
@amattu2 amattu2 added this to the 3.2.0 (PMVP-M3) milestone Jan 22, 2025
@coveralls
Copy link
Collaborator

coveralls commented Jan 23, 2025

Pull Request Test Coverage Report for Build 12952999958

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 52 of 61 (85.25%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.4%) to 58.326%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/content/questionnaire/ListView.tsx 5 14 35.71%
Totals Coverage Status
Change from base Build 12933185675: 0.4%
Covered Lines: 4079
Relevant Lines: 6567

💛 - Coveralls

@amattu2 amattu2 marked this pull request as ready for review January 23, 2025 16:42
@amattu2 amattu2 removed the 🚧 Do Not Merge This PR is not ready for merging label Jan 24, 2025
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.

2 participants