Skip to content

New endpoint to unbuild all nodes of a study#900

Merged
klesaulnier merged 8 commits intomainfrom
unbuild-all-nodes
Dec 17, 2025
Merged

New endpoint to unbuild all nodes of a study#900
klesaulnier merged 8 commits intomainfrom
unbuild-all-nodes

Conversation

@klesaulnier
Copy link
Contributor

PR Summary

All nodes from all root networks of a study will be unbuilt if this endpoint is called

klesaulnier and others added 2 commits December 12, 2025 14:11
Signed-off-by: LE SAULNIER Kevin <kevin.lesaulnier.pro@gmail.com>
@SlimaneAmar SlimaneAmar self-requested a review December 16, 2025 12:34
return ResponseEntity.ok().build();
}

@PostMapping(value = "/studies/{studyUuid}/nodes/unbuild-all")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

POST o unbuild ?

Perhaps supervision path is better
@DeleteMapping(value = "/studies/{studyUuid}/nodes/builds")

Signed-off-by: LE SAULNIER Kevin <kevin.lesaulnier.pro@gmail.com>
Signed-off-by: LE SAULNIER Kevin <kevin.lesaulnier.pro@gmail.com>
return getNodeInfoEntity(nodeUuid).getReadOnly();
}

public Boolean isRootNode(UUID nodeUuid) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace by primitive type : boolean
Fix Sonar warning

Signed-off-by: LE SAULNIER Kevin <kevin.lesaulnier.pro@gmail.com>
Signed-off-by: LE SAULNIER Kevin <kevin.lesaulnier.pro@gmail.com>
@sonarqubecloud
Copy link

@klesaulnier klesaulnier merged commit ac2eec5 into main Dec 17, 2025
4 checks passed
@klesaulnier klesaulnier deleted the unbuild-all-nodes branch December 17, 2025 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants