-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feature/value updater #440
Conversation
This pull request introduces 1 alert when merging e4fa254 into 8515f05 - view on LGTM.com new alerts:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks.
We'll need to update the RabbitMQ configs and architecture diagrams once we have the other bits of this pipeline in place, but this PR is good for now.
src/microservices/Updating/Microservices.UpdateValues/Execution/UpdateValuesHost.cs
Show resolved
Hide resolved
src/microservices/Updating/Microservices.UpdateValues/Execution/UpdateValuesQueueConsumer.cs
Show resolved
Hide resolved
This pull request introduces 1 alert when merging 897d317 into 8515f05 - view on LGTM.com new alerts:
|
Proposed Changes
Added new service UpdateValues which propagates changes (e.g. ECHI mapping changes) throughout the deployed database tables.
Types of changes
What types of changes does your code introduce? Tick all that apply.
[skip ci]
Checklist
By opening this PR, I confirm that I have: