Closed MontaltoNick closed 1 week ago
The changes involve modifications to the QueryPlan
class, including the removal of the ReportType
property and the introduction of a new Type
property of the QueryPlanType
enumeration. Additionally, several migration files were created or updated to reflect these changes in the database schema. Other adjustments include updates to various services and models to accommodate the new structure, as well as the introduction of properties related to reportable events in multiple classes.
Files | Change Summary |
---|---|
DotNet/DataAcquisition.Domain/Entities/QueryPlan.cs |
Removed ReportType , added Type (of type QueryPlanType ). |
DotNet/DataAcquisition.Domain/Migrations/*.cs |
Created migration to remove ReportType and add Type (int). |
DotNet/DataAcquisition.Domain/Migrations/DataAcquisitionDbContextModelSnapshot.cs |
Updated model snapshot to reflect removal of ReportType and addition of Type . |
DotNet/DataAcquisition.Domain/Models/QueryPlanType.cs |
Introduced QueryPlanType enumeration with members Monthly , Weekly , Daily , Discharge . |
DotNet/DataAcquisition/Application/Managers/QueryPlanManager.cs |
Updated UpdateAsync method to use Type instead of ReportType . |
DotNet/DataAcquisition/Application/Services/PatientDataService.cs |
Modified logic to compare Type instead of ReportType . |
DotNet/Normalization/Application/Models/Messages/*.cs |
Added ReportableEvent property to ResourceAcquiredMessage and ResourceNormalizedMessage . |
DotNet/Normalization/Application/Models/Messages/ScheduledReport.cs |
Replaced ReportType with ReportTypes (array of strings) and added Frequency . |
DotNet/QueryDispatch/Application/Models/*.cs |
Added ReportableEvent to DataAcquisitionRequestedValue and introduced Frequency enumeration. |
DotNet/QueryDispatch/Application/Factory/QueryDispatchFactory.cs |
Updated CreateScheduledReport method to include a new frequency parameter. |
DotNet/QueryDispatch/Application/Interfaces/IQueryDispatchFactory.cs |
Updated method signature for CreateScheduledReport to include frequency . |
DotNet/QueryDispatch/Application/Extensions/KafkaRegistration.cs |
Changed key type from ReportScheduledKey to string in Kafka consumer and producer factories. |
DotNet/Normalization/Listeners/ResourceAcquiredListener.cs |
Added validation for ReportableEvent and included it in audit event parameters. |
Objective | Addressed | Explanation |
---|---|---|
Merge Dev Into Multi Measure (LNK-3015) | โ |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
๐ Description of Changes
Please provide a high-level overview of the changes included in this PR.
๐งช Testing Performed
Please describe the testing that was performed on the changes included in this PR.
Summary by CodeRabbit
Release Notes
New Features
QueryPlanType
enumeration to categorize query plans (Monthly, Weekly, Daily, Discharge).ReportableEvent
property to multiple message classes for enhanced resource acquisition context.ScheduledReport
to support multiple report types and introduced aFrequency
property.Frequency
enumeration added to represent frequency options (Daily, Weekly, Monthly).Bug Fixes
PatientDataService
to ensure correct matching ofqueryPlan
types.Improvements
CreateScheduledReport
method to include frequency as a parameter for better scheduling flexibility.