{RW-8595][risk=no] remove code referencing enable drug wildcard search #6996
Conversation
| "enablePrivateDataprocWorker": true, | ||
| "ccSupportWhenAdminLocking": false, | ||
| "enableMultiReview": true, | ||
| "enableDrugWildcardSearch": true, |
There was a problem hiding this comment.
Story only says to remove references to flag, but not the flag itself. Please revert.
| "enablePrivateDataprocWorker": true, | ||
| "ccSupportWhenAdminLocking": false, | ||
| "enableMultiReview": false, | ||
| "enableUpdatedDemographicSurvey": true, |
There was a problem hiding this comment.
Story only says to remove references to flag, but not the flag itself. Please revert.
| "ccSupportWhenAdminLocking": true, | ||
| "enableMultiReview": false, | ||
| "enableUpdatedDemographicSurvey": true, | ||
| "enableDrugWildcardSearch": true |
There was a problem hiding this comment.
Story only says to remove references to flag, but not the flag itself. Please revert.
| "ccSupportWhenAdminLocking": true, | ||
| "enableMultiReview": false, | ||
| "enableUpdatedDemographicSurvey": true, | ||
| "enableDrugWildcardSearch": true |
There was a problem hiding this comment.
Story only says to remove references to flag, but not the flag itself. Please revert.
| "ccSupportWhenAdminLocking": false, | ||
| "enableMultiReview": false, | ||
| "enableUpdatedDemographicSurvey": true, | ||
| "enableDrugWildcardSearch": true |
There was a problem hiding this comment.
Story only says to remove references to flag, but not the flag itself. Please revert.
| "ccSupportWhenAdminLocking": false, | ||
| "enableMultiReview": false, | ||
| "enableUpdatedDemographicSurvey": true, | ||
| "enableDrugWildcardSearch": true |
There was a problem hiding this comment.
Story only says to remove references to flag, but not the flag itself. Please revert.
| "ccSupportWhenAdminLocking": false, | ||
| "enableMultiReview": true, | ||
| "enableUpdatedDemographicSurvey": true, | ||
| "enableDrugWildcardSearch": true |
There was a problem hiding this comment.
Story only says to remove references to flag, but not the flag itself. Please revert.
| public boolean enableDrugWildcardSearch; | ||
| // If true, enable Drug Wildcard Search for Cohort Builder - backend |
There was a problem hiding this comment.
Story only says to remove references to flag, but not the flag itself. Please revert.
| @Mapping( | ||
| target = "enableDrugWildcardSearch", | ||
| source = "config.featureFlags.enableDrugWildcardSearch") |
There was a problem hiding this comment.
Story only says to remove references to flag, but not the flag itself. Please revert.
| enableDrugWildcardSearch: | ||
| type: boolean | ||
| default: false | ||
| description: Whether to enable wildcard search for Durgs in Cohort Builder |
There was a problem hiding this comment.
Story only says to remove references to flag, but not the flag itself. Please revert.
| serverConfigStore.set({ | ||
| config: { | ||
| ...defaultServerConfig, | ||
| enableDrugWildcardSearch: false, | ||
| }, | ||
| }); |
There was a problem hiding this comment.
Since the component no longer references serverConfigStore we can remove it from the test also.
| serverConfigStore.set({ | ||
| config: { | ||
| ...defaultServerConfig, | ||
| enableDrugWildcardSearch: false, | ||
| }, | ||
| }); |
There was a problem hiding this comment.
Same as above, can remove serverConfigStore from the test.
| serverConfigStore.set({ | ||
| config: { | ||
| ...defaultServerConfig, | ||
| enableDrugWildcardSearch: false, | ||
| }, | ||
| }); |
There was a problem hiding this comment.
Can remove serverConfigStore from the test.
| serverConfigStore.set({ | ||
| config: { | ||
| ...defaultServerConfig, | ||
| enableDrugWildcardSearch: false, | ||
| }, | ||
| }); |
There was a problem hiding this comment.
Can remove serverConfigStore from the test.
| serverConfigStore.set({ | ||
| config: { | ||
| ...defaultServerConfig, | ||
| enableDrugWildcardSearch: false, | ||
| }, | ||
| }); |
There was a problem hiding this comment.
Can remove serverConfigStore from the test.
|
Closing this PR as the changes are not in sync with my understanding of the requirements of the story. |
Description:
PR checklist