Closed MontaltoNick closed 1 week ago
The pull request involves the complete removal of several files related to tenant submission management, including the ITenantSubmissionManager
interface and its implementing class TenantSubmissionManager
. Additionally, multiple classes and interfaces associated with authentication and database context management have been deleted. The changes streamline the codebase by eliminating unused or unnecessary components, impacting the overall structure of the submission service.
File Path | Change Summary |
---|---|
DotNet/Submission/Application/Interfaces/ITenantSubmissionManager.cs |
Interface ITenantSubmissionManager removed. |
DotNet/Submission/Application/Managers/TenantSubmissionManager.cs |
Class TenantSubmissionManager and its methods removed. |
DotNet/Submission/Application/Models/ApiModels/Method.cs |
Class Method removed. |
DotNet/Submission/Application/Models/CensusAdmittedPatient.cs |
Class CensusAdmittedPatient removed. |
DotNet/Submission/Application/Models/FileSystemConfig.cs |
Class FileSystemConfig removed. |
DotNet/Submission/Application/Services/Auth/BasicAuth.cs |
Class BasicAuth and its method SetAuthentication removed. |
DotNet/Submission/Application/Services/Interfaces/IAuth.cs |
Interface IAuth and its method SetAuthentication removed. |
DotNet/Submission/Domain/Entities/TenantSubmissionConfigEntity.cs |
Class TenantSubmissionConfigEntity removed. |
DotNet/Submission/Domain/Models/AuthenticationConfiguration.cs |
Class AuthenticationConfiguration removed. |
DotNet/Submission/Domain/SubmissionEntityRepository.cs |
Class SubmissionEntityRepository<T> removed. |
DotNet/Submission/Domain/TenantSubmissionDbContext.cs |
Class TenantSubmissionDbContext removed. |
DotNet/Submission/Infrastructure/SubmissionHealthCheck.cs |
Class SubmissionHealthCheck and its method CheckHealthAsync removed. |
DotNet/Submission/Listeners/SubmitReportListener.cs |
Unused using directives removed. |
DotNet/Submission/Migrations/20240610170909_202406101308_Initial.Designer.cs |
Migration file removed. |
DotNet/Submission/Migrations/20240610170909_202406101308_Initial.cs |
Migration class removed. |
DotNet/Submission/Migrations/TenantSubmissionDbContextModelSnapshot.cs |
Model snapshot removed. |
DotNet/Submission/Program.cs |
Service registrations and configurations for database context, health checks, and Swagger removed. |
DotNet/Submission/Settings/SubmissionConstants.cs |
Constants DatabaseProvider and DatabaseConnectionString removed. |
DotNet/Submission/Submission.csproj |
Removed entries for Domain\Interfaces from project file. |
Objective | Addressed | Explanation |
---|---|---|
Refactor submission service to use ServiceRegistry (LNK-2507) | โ | The pull request does not indicate any refactoring to use ServiceRegistry. |
TenantSubmissionManager
and the ITenantSubmissionManager
interface, directly related to their removal in this PR.ReportableEvent
in various classes, potentially impacting data handling related to tenant submissions.
๐ Description of Changes
Please provide a high-level overview of the changes included in this PR.
Cleanup unused DB items, classes, old authorization files, configs, data models, etc.
๐งช Testing Performed:
Please describe the testing that was performed on the changes included in this PR.
Service start up and Kafka Message Receivable to SubmitReportListener
Summary by CodeRabbit
New Features
Bug Fixes
Refactor
Chores