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

PEPPER-1388 remove angio from pancan redirect #2933

Merged
merged 4 commits into from
Sep 11, 2024

Conversation

ssettipalli
Copy link
Contributor

PEPPER-1388
remove angio study redirect from pancan now that angio enrollment is ending.

  1. Remove/Delete the study_redirect workflow transition for Angiosarcoma
  2. Several prequal pex expressions have "C_SARCOMAS_ANGIOSARCOMA" which check various conditions. Remove this from all pex expressions. No other study has this String C_SARCOMAS_ANGIOSARCOMA, so doing global search and replacing with a pancan related where clause.
    below are the overall category of pex expressions which has all different study redirect substitutions. search and replace will cover all of these except has_angio. workflow_transition deletion will make has_angio obsolete (no usages)
    "has_angio": """
    has_cmi_legacy
    is_not_redirect
    is_redirect

Verified locally applying the patch and looked fine.

  1. StudyBuilder pancan subs.conf is just to keep config in sync with what patch is doing and patch doest use this file.

@@ -177,7 +177,7 @@
)
||
(user.studies["cmi-pancan"].forms["PREQUAL"].questions["PRIMARY_CANCER_LIST_CHILD"].numChildAnswers("PRIMARY_CANCER_CHILD") > 1
&& user.studies["cmi-pancan"].forms["PREQUAL"].questions["PRIMARY_CANCER_LIST_CHILD"].children["PRIMARY_CANCER_CHILD"].answers.hasOptionStartsWith("C_SARCOMAS_ANGIOSARCOMA", "C_GASTRO_ESOPHAGEAL_CANCER", "C_GASTRO_GASTRIC_STOMACH_CANCER", "C_GENITOURINARY_PROSTATE", "C_BREAST_")
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file changes are just to keep config in sync with what patch is doing and patch doest use this file.

@ssettipalli
Copy link
Contributor Author

@zyme
When you get few minutes today, can you please take a look at this PR. Thanks.

@ssettipalli ssettipalli merged commit ada39dc into develop Sep 11, 2024
8 checks passed
@ssettipalli ssettipalli deleted the PEPPER-1388-remove-pancan-redirect branch September 11, 2024 18:56
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