sequencescape icon indicating copy to clipboard operation
sequencescape copied to clipboard

Y24-058 [BUG] Study data release information - data clean up

Open KatyTaylor opened this issue 10 months ago • 0 comments

Describe the bug During development of https://github.com/sanger/sequencescape/issues/4076, I found that many of the existing Studies in the Sequencescape database do not conform to the new validation rules. Because of this, I deployed the new server-side validation for that story 'feature flagged off'. This story is to track the resolution of this. Ideally, we can 'fix' all the existing studies, so that they pass the new validation, and then remove the feature flag.

There is an ongoing conversation via email between Katy, Liz C and Governance as to whether the validation rules are correct, and whether we can 'fix' those studies.

Additional context See query to find records that fail the validation here: https://github.com/sanger/sequencescape/issues/4076#issuecomment-2045269998

Table that shows the expected relationships between fields, which the new validation would enforce:

Data release strategy Data release timing
Managed or Open Standard, Immediate or Delayed
Not applicable Never

To remove the feature flag, merge the following pull request: https://github.com/sanger/sequencescape/pull/4086 This will be the first time the server-side validation is enabled, so consider doing some extra testing of it.

KatyTaylor avatar Apr 17 '24 13:04 KatyTaylor