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.

Sign up for GitHub

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

Jump to bottom

feat: Start MeteredDataForMeasurementPoint orchestration from EDI #1425

Open
wants to merge 30 commits into
base: main
Choose a base branch
from
Open

feat: Start MeteredDataForMeasurementPoint orchestration from EDI #1425

wants to merge 30 commits into from

Conversation

AndersBallingPetersen
Copy link
Contributor

Description

Before:
MeteredDataForMeasurementPoint created the RSM-012 message directly in EDI without interaction with the ProcessManager.

Now:
MeteredDataForMeasurementPoint starts a new orchestration using the ProcessManagerClient.

FYI: The Featureflag for MeteredDataForMeasurementPoint is checked in Energinet.DataHub.EDI.B2BApi.IncomingMessages.IncomingMessageReceiver

References

Checklist

  • Should the change be behind a feature flag?
  • Can the feature be meaningfully disabled or circumvented if there are issues (e.g., database-breaking changes)?
  • Has it been considered whether data is being delivered to the wrong actor?
  • Subsystem test executed (dev_002/dev_003)
  • Is there time to monitor state of the release to Production?
  • Reference to the task

ebbeknudsen and others added 20 commits December 11, 2024 12:05
@ebbeknudsen
Send Wholesale/Energi requests to PM
6e1d063 
- Add ActorId to authenticated actor
- Use ProcessManager.Client to start processes in PM
@ebbeknudsen
Update EDI to start process for each transaction
fba009a
@ebbeknudsen
Update RequestProcessOrchestrationStarter.cs
8ff7e91
@ebbeknudsen
Fix architecture tests
165a6b4
@ebbeknudsen
Move process manager call to incoming messages
77e82f2
@ebbeknudsen
Fix test
0a62753
@ebbeknudsen
Fix B2C tests
95a25f8
@ebbeknudsen
Fix incoming messages test
d03b164
@ebbeknudsen
Default to an "empty" spy if none is registered
7bd7171
@ebbeknudsen
Update to use a default spy instead of a new spy
afa6be9
@ebbeknudsen
Add missing health check
0c45187
@ebbeknudsen
Use actual values for starting process
edad36a
@ebbeknudsen
Add test for mapping
b2a2924
@ebbeknudsen
Add app test for starting energy request
9a1d83c
@ebbeknudsen
Fix test
f3003b2
@ebbeknudsen
Update RequestAggregatedMeasureDataTests.cs
46884d2
@AndersBallingPetersen
Wip
d498803
@AndersBallingPetersen
Merge branch 'main' into xabpe/Start-from-Command-in-EDI
6f3520e
@AndersBallingPetersen
Start MeteredDataForMeasurementPoint orchestration from EDI
98e6d23
@AndersBallingPetersen
Merge branch 'main' into xabpe/Start-from-Command-in-EDI
9a5c787
@AndersBallingPetersen AndersBallingPetersen requested a review from a team as a code owner December 18, 2024 07:50
@AndersBallingPetersen AndersBallingPetersen force-pushed the xabpe/Start-from-Command-in-EDI branch from 5afafbd to eb0aa7d Compare January 6, 2025 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MWO1024 MWO1024 MWO1024 requested changes

@MadsDue MadsDue MadsDue left review comments

@RSIEnerginet RSIEnerginet Awaiting requested review from RSIEnerginet RSIEnerginet is a code owner automatically assigned from Energinet-DataHub/team-mosaic

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AndersBallingPetersen @MadsDue @MWO1024 @ebbeknudsen