CARDS-2026: Automatically delete names and emails once they are no longer serving any purpose #1305
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
veronikaslc
force-pushed
the
CARDS-2026
branch
from
a0db4a9
to
50195da
Compare
marta-
added
the
needs test instructions
|
@veronikaslc this doesn't look like it would depend on |
marta-
reviewed
Feb 5, 2023
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
marta-
reviewed
Feb 6, 2023
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
veronikaslc
force-pushed
the
CARDS-2026
branch
from
25918cc
to
280ff06
Compare
marta-
added
the
needs refactoring
|
@veronikaslc The specs to https://phenotips.atlassian.net/browse/CARDS-2026 have been updated to make use of the implementation of the |
marta-
reviewed
Feb 13, 2023
...tal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupScheduler.java
Outdated
Show resolved
Hide resolved
sdumitriu
reviewed
Feb 14, 2023
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
veronikaslc
force-pushed
the
CARDS-2026
branch
from
44974d4
to
7460436
Compare
sdumitriu
reviewed
Feb 15, 2023
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
veronikaslc
force-pushed
the
CARDS-2026
branch
from
250e2b1
to
bd362ff
Compare
veronikaslc
removed
the
needs refactoring
marta-
reviewed
Feb 16, 2023
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
|
I've added a configuration for disabling this, and I also moved the |
sdumitriu
requested changes
Jul 13, 2023
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
...tal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupScheduler.java
Outdated
Show resolved
Hide resolved
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
...t-portal/src/main/java/io/uhndata/cards/patients/internal/PatientInformationCleanupTask.java
Outdated
Show resolved
Hide resolved
veronikaslc
commented
Jul 14, 2023
...-portal/src/main/java/io/uhndata/cards/patients/internal/DataRetentionConfigurationImpl.java
Outdated
Show resolved
Hide resolved
sdumitriu
reviewed
Jul 22, 2023
...-portal/src/main/java/io/uhndata/cards/patients/internal/DataRetentionConfigurationImpl.java
Outdated
Show resolved
Hide resolved
modules/patient-portal/src/main/frontend/src/patient-portal/DataRetentionConfiguration.jsx
Outdated
Show resolved
Hide resolved
modules/patient-portal/src/main/frontend/src/patient-portal/DataRetentionConfiguration.jsx
Outdated
Show resolved
Hide resolved
modules/patient-portal/src/main/frontend/src/patient-portal/DataRetentionConfiguration.jsx
Outdated
Show resolved
Hide resolved
…nger serving any purpose
…nger serving any purpose Fixed sql query; Better sql query readability
…nger serving any purpose Fixed version and query args
…nger serving any purpose Only remove the value from the answer, not the whole answer node
…nger serving any purpose Refactored code wrt changed issue description: First fetch all Patient information forms for which all Servey events have non null discharged_date Then for each Patient information form fetch all associated Servey event forms in desc visit order Look at the latest Servey event form to see if it has at least one non null of responses_received or reminder2_sent answers
…nger serving any purpose Refactored to delete pationt information if there is no Survey events form Fixed always returning false
…nger serving any purpose Search only for the patient information forms that do have any private information to remove
…nger serving any purpose Bugfix: checked in forms cannot be cleared
…nger serving any purpose Fixed typo in reminder2_sent field Co-authored-by: Marta Girdea <[email protected]>
…nger serving any purpose Speed up the query
…nger serving any purpose Reduced cyclomatic complexity
…nger serving any purpose Add configuration for enabling/disabling the task
…onger serving any purpose Fixed comments
…onger serving any purpose Added a new data retension admin config screen
…nger serving any purpose Change the configuration path to /apps/cards/config/DataRetention Minor comment fixes
…nger serving any purposeUpdate modules/patient-portal/src/main/frontend/src/patient-portal/DataRetentionConfiguration.jsx Better wording for labels Co-authored-by: Sergiu Dumitriu <[email protected]>
veronikaslc
force-pushed
the
CARDS-2026
branch
from
2092188
to
4747577
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.