merge develop -> main#53
Merged
Merged
Conversation
* Refactor: migrate from .env file to standard .NET configuration
- Remove DotNetEnv package dependency and development.local.env file
- Move configuration to appsettings.json with environment variable overrides
- Consolidate logging setup with UseDmsSerilog() extension method
- Create AddDmsCoreWorkerService() bundle for common worker service configuration
- Simplify RabbitMQ configuration to use connection strings only
- Remove ConnectionStrings wrapper class in favor of standard GetConnectionString()
- Standardize Serilog file paths to use Path.Combine for cross-platform compatibility
- Add try/catch/finally error handling to Meow Program.cs
- Clean up redundant configuration registrations across all projects
* Cleanup connection string usage and remove unused files
* Add placeholders for required ConnectionStrings
* Remove unused launchSettings for non ASP.NET apps
* Remove unused docker configurations
* Include RabbitMQ clarification for Meow
* Cleanup project references
* Make Aspire service naming consistent
* Remove .env file
* Add Messaging.Tests project Includes Testcontainers project for end to end Rabbit message testing * WIP: refactor RabbitService, use single Publish/Subscribe method * Refactor DMS messaging. Use single Publish/Subscribe methods. * Remove StatusPublish/StatusSubscribe and use generic Publish/Subscribe * Remove IDbMessagingService and replace with generic IMessageService * Rename messages and use DbRequestMessage<TResponse> * Ensure RabbitService serializes generic type to concrete implementation * Remove constructors and rely on properties for initialisation Migrates from constructor-based to property-based initialization for message payloads, ensuring consistency in data transfer between services.
Renames the RabbitMQ configuration setting to CatsRabbitMQ to clearly indicate that Meow uses the CATS RabbitMQ instance.
Configures the application's default settings for file synchronization.
* Fixes for Sentry configuration * Sets current directory for Windows services Resolves an issue with relative paths when logging to a file * Updates log file paths Updates the log file paths in the application settings files to use relative paths, ensuring logs are stored in a consistent location.
Configures the project to support the Aspire Visual Studio Code extension.
* Remove explicit start from FileSync * Enables data seeding and service control * Amend seeding script The seeding procedure will now first checks if the Nomis or Delius records already exist to prevent duplicates. The script ensures idempotency of the process. * Remove redundant waits * Update CreateDummyCandidate ClusterId generation * Tweak identifiers that clashed with pre-seeded clusters
Contributor
Author
|
Changes has been verified in PreProduction. Ready for deployment to Production tomorrow. cc: Carl Sixsmith (@carlsixsmith-moj) Ryan Kearsley (@ryankearsley-moj) |
Ryan Kearsley (ryankearsley-moj)
approved these changes
Dec 5, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.