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

Add download handler #22

Merged
merged 6 commits into from
Sep 16, 2024
Merged

Add download handler #22

merged 6 commits into from
Sep 16, 2024

Conversation

FirestarJes
Copy link
Contributor

@FirestarJes FirestarJes commented Sep 13, 2024

Description

Pull-request quality

  • The title adheres to this guide
  • Tests are written and executed locally
  • Subsystem tests have been tested (by manually deploying to dev_002)

Copy link

github-actions bot commented Sep 13, 2024

Test results for \source\settlement-report\Orchestration.SettlementReports.IntegrationTests\bin\Release\net8.0\Energinet.DataHub.SettlementReport.Orchestration.SettlementReports.IntegrationTests.dll

4 tests   4 ✅  36s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit 2cfdc99.

♻️ This comment has been updated with latest results.

also add missing variables for integration test
Also ensure that the func list method only updates status of non job reports
@FirestarJes FirestarJes merged commit d9e97ce into main Sep 16, 2024
18 checks passed
@FirestarJes FirestarJes deleted the juj/AddDownloadReportToWebAPI branch September 16, 2024 11:52
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