Skip to content

Resolve "Feature flag for CNVs"

Øyvind Evju requested to merge 1937-cnv-feature-flag into 56-cnv-minimal-rc1

Description

Add feature flag to application config, setup to read ENABLE_CNV from environment. Defaults to false, but all dev and test environments have been set up with CNV enabled.

When feature flag is disabled it will:

  • Stop deposit of CNV alleles
  • Hide button group used to changing between SNV and CNV view
  • Skip CNV data when importing test data

Related issues

Notes to review (code/docs/QA)

Are there other places we need this feature flag?

Tests

General

  • Tests have been added that prove my fix is effective or that my feature works
  • Related tests have been modified/removed

Hypothesis testing:

  • Soak testing has been done
  • Distribution between positive / negative cases has been checked

Database

  • Includes changes to database schema
  • Includes necessary database migrations

Configuration

  • Includes changes to configuration
  • Includes configuration migration instructions in documentation

Merge checklist

  • Self-review of code has been performed.
  • Feature review and validation against specification has been performed (if applicable). Apply label: QAdone
  • Need for documentation has been evaluated and, if necessary, updated. Apply label: docsdone
  • Code and implementation has been reviewed by other core developer (including any changes based on initial review). Apply label: code reviewdone

Closes #1937 (closed)

Edited by Tom

Merge request reports