powsybl / powsybl-afs

AFS modules for powsybl
Mozilla Public License 2.0
3 stars 1 forks source link

Ajout du node ID dans le start Task #99

Closed Lunaria closed 3 years ago

Lunaria commented 3 years ago

Signed-off-by: quentin quentin.capy@rte-france.com

Please check if the PR fulfills these requirements (please use '[x]' to check the checkboxes, or submit the PR and then click the checkboxes)

Does this PR already have an issue describing the problem ? If so, link to this issue using '#XXX' and skip the rest issue from Imagrid team who need the node id of a task to link the task notification to the afs node

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) A new field is available on StartTaskEvent to get the nodeId link to this task

What is the current behavior? (You can also link to an open issue here) no link possible between task and afs node with task notification

What is the new behavior (if this is a feature change)? the provided nodeID link the task and afs node during startTask notification

Does this PR introduce a breaking change or deprecate an API? If yes, check the following:

Other information: there is no breaking change on the backend part. All listener should check if they check explicitly all tag of this event "StartTaskEvent" in order to add the possibility to find the nodeId (if any of the questions/checkboxes don't apply, please delete them entirely)

sonarcloud[bot] commented 3 years ago

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

69.2% 69.2% Coverage
0.0% 0.0% Duplication