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

Dfpl 2439 query collections #5582

Open
wants to merge 101 commits into
base: master
Choose a base branch
from
Open

Conversation

jamesrferguson1
Copy link
Contributor

JIRA link (if applicable)

https://tools.hmcts.net/jira/browse/DFPL-2439

Change description

Adding query collections for case management.

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x ] No

@jamesrferguson1 jamesrferguson1 requested a review from a team as a code owner September 11, 2024 16:18
@github-actions github-actions bot added the ccd configuration Pull request that updates CCD definition configuration label Sep 11, 2024
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview September 11, 2024 16:32 Abandoned
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview September 13, 2024 09:51 Abandoned
updating image of xui-webapp for testing.
updating correct xui-mc image for QA
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 27, 2025 08:39 Abandoned
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 27, 2025 10:22 Abandoned
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 27, 2025 10:48 Abandoned
@qasimnawaz-moj qasimnawaz-moj added pr-values:wa Enables WA in preview and removed enable_wa pr-values:wa Enables WA in preview labels Jan 27, 2025
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 27, 2025 12:32 Abandoned
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 28, 2025 10:42 Abandoned
@@ -53,7 +53,7 @@ xui-webapp:
imagePullPolicy: Always
replicas: 1
releaseNameOverride: ${SERVICE_NAME}-xui-webapp
image: hmctspublic.azurecr.io/xui/webapp:latest
image: hmctspublic.azurecr.io/xui/webapp:pr-4140-dc9226f-20250113180116
Copy link
Contributor Author

Choose a reason for hiding this comment

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

guess will need to remember to take this out when we're done testing

@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 28, 2025 12:26 Abandoned
Copy link
Contributor

@DanCatchpole DanCatchpole left a comment

Choose a reason for hiding this comment

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

all good, just a couple things that must be removed before deployment!

Comment on lines +71 to +76
private void logQueryCollection(CaseDetails caseDetails, String collectionKey) {
if (caseDetails.getData().containsKey(collectionKey)) {
log.info("{} is present with value: {}", collectionKey, caseDetails.getData().get(collectionKey));
} else {
log.info("{} is not present on case data.", collectionKey);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

logging will need to be removed before deployment, as field contents may contain PII.

@@ -156,7 +156,7 @@ def deployWADmns() {

def waStandaloneBranch = "master"
// For testing DMNs, change this to the branch on the DMN repo, but ensure it is changed back to master before merging
def dmnBranch = "master"
def dmnBranch = "DFPL-2439-v2"
Copy link
Contributor

Choose a reason for hiding this comment

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

just a note to remove before deployment!

@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 29, 2025 10:54 Abandoned
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 29, 2025 11:34 Abandoned
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 29, 2025 12:42 Abandoned
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants