Skip to content

Fix Version Table Archiving Submit logic#12104

Open
qqmyers wants to merge 3 commits intoIQSS:developfrom
QualitativeDataRepository:Arch4-fix_isArchivable_caching
Open

Fix Version Table Archiving Submit logic#12104
qqmyers wants to merge 3 commits intoIQSS:developfrom
QualitativeDataRepository:Arch4-fix_isArchivable_caching

Conversation

@qqmyers
Copy link
Member

@qqmyers qqmyers commented Jan 21, 2026

What this PR does / why we need it: The Dataset Page Version table shows info about the archival state of dataset versions (when archiving is used) and shows a submit button for superadmins to resubmit unarchived versions. However, the caching logic to decide whether a version can be archived (and therefore what to show in the table) was only designed for a single version and therefore caused info to display incorrectly when multiple versions exist.
This PR corrects the caching to be per version and updates the display to use the new per version info.

Which issue(s) this PR closes:

  • Closes #

Special notes for your reviewer:
FWIW: This is one of several PRs related to updates being done to improve archiving scalability at QDR and DANS. Some are relatively minor cleanup while others are more significant. I'm submitting them all to avoid drift in QDR's fork and to leverage QDR testing (where all of these are being tested together).

Suggestions on how to test this: Could configure archiving per the guides but, since the display is run based on the recorded status in the datasetversion object, it may be easier to add status to multiple versions to show the difference. I think one simple case would be to have a v2 that hasn't been archived and a v1 that was successfully archived. With the current code, both versions will show/not show the submit button for a superuser. With the PR, submit should only be shown for the one where archiving was not successful.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?: Will create an aggregate one

Additional documentation:

@coveralls
Copy link

coveralls commented Jan 21, 2026

Coverage Status

coverage: 24.333% (-0.001%) from 24.334%
when pulling 69dee01 on QualitativeDataRepository:Arch4-fix_isArchivable_caching
into cc37d4d on IQSS:develop.

@qqmyers qqmyers marked this pull request as ready for review January 28, 2026 20:27
@qqmyers qqmyers added GDCC: DANS related to GDCC work for DANS GDCC: QDR of interest to QDR Size: 3 A percentage of a sprint. 2.1 hours. labels Jan 28, 2026
@qqmyers qqmyers added this to the 6.10 milestone Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GDCC: DANS related to GDCC work for DANS GDCC: QDR of interest to QDR Size: 3 A percentage of a sprint. 2.1 hours.

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants