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.

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

Fix creating and editing Fault Tree #95

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

kostobog
Copy link
Collaborator

@LaChope
This PR fixes backend API for creating and editing fault tree.

@kostobog kostobog force-pushed the fix/fix-fault-tree-creation-and-editing-api branch 3 times, most recently from c9303c9 to b3865d0 Compare May 17, 2024 19:43
@blcham
Copy link
Contributor

blcham commented May 18, 2024

@LaChope it would be nice to rebase this PR as well before merging

@kostobog kostobog force-pushed the fix/fix-fault-tree-creation-and-editing-api branch 2 times, most recently from a9c0624 to 9dfdb41 Compare May 21, 2024 07:34
@kostobog kostobog changed the title Change FaultEvent.isReference from boolean to Boolean. Fix creating and editing Fault Tree May 21, 2024
@blcham
Copy link
Contributor

blcham commented May 22, 2024

@kostobog is this finished?

@kostobog kostobog force-pushed the fix/fix-fault-tree-creation-and-editing-api branch from 9dfdb41 to 2ed68a2 Compare June 4, 2024 13:53
- update BaseDao.getContext to not fail when entity is in multiple contexts.
- add FaultEventDao.findEvent
- fix FaultTreeRepositoryService.createTree to replace root supertypes for managed entities.
@blcham blcham force-pushed the fix/fix-fault-tree-creation-and-editing-api branch from 2ed68a2 to 88621d6 Compare June 4, 2024 14:05
@blcham blcham merged commit fc939e5 into main Jun 4, 2024
1 check passed
blcham pushed a commit that referenced this pull request Jun 4, 2024
- update BaseDao.getContext to not fail when entity is in multiple contexts.
- add FaultEventDao.findEvent
- fix FaultTreeRepositoryService.createTree to replace root supertypes for managed entities.
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