Skip to content

Fact 2634 csv creation for cron job#232

Open
RebeccaHayleyPickles wants to merge 56 commits into
masterfrom
FACT-2634-Csv-creation-for-cron-job
Open

Fact 2634 csv creation for cron job#232
RebeccaHayleyPickles wants to merge 56 commits into
masterfrom
FACT-2634-Csv-creation-for-cron-job

Conversation

@RebeccaHayleyPickles

Copy link
Copy Markdown
Contributor

FACT-2634

Add csv creation and update from court data and add slack alert if there are any problems

Does this PR require manual testing? (check one with "x")

[x] Yes
[ ] No

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

[ ] Yes
[x] No

@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i Bot requested a deployment to preview May 13, 2026 13:53 Abandoned
@RebeccaHayleyPickles RebeccaHayleyPickles marked this pull request as ready for review May 13, 2026 14:40
Comment thread src/main/java/uk/gov/hmcts/reform/fact/data/api/models/StringMultipartFile.java Outdated
Comment thread src/main/java/uk/gov/hmcts/reform/fact/data/api/models/StringMultipartFile.java Outdated
Comment thread src/main/java/uk/gov/hmcts/reform/fact/data/api/services/AzureBlobService.java Outdated

@amstevenson amstevenson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Need to perhaps to the infra as well for this to add the new container

Comment thread src/main/java/uk/gov/hmcts/reform/fact/data/api/services/CsvService.java Outdated

@amstevenson amstevenson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Perhaps check the application yaml, as another container may be needed there. Have you checked this works locally also against the local storage account?

@sonarqubecloud

Copy link
Copy Markdown

@sonarqubecloud

Copy link
Copy Markdown

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.

2 participants