Actions
Architecture #3557
closedRefactoring in Workflows
Status:
Rejected
Priority:
3
Assignee:
-
Category:
Architecture - Code maintenance
Target version:
Pull Request:
Effort required:
Name check:
Fix check:
Regression:
No
Description
Now that workflows are more or less stabillized, we have a far better view about what we need and what should desapear.
Some requirement not fulfullid:
- "workflow enabled" properties should be use at one point only
- in [WR]oChangeRequestJdbcRepository, we should not use Try to immediatly test for success
- we don't want to use Box.Empty
- in other data access layer, logs are done in the repository, not in the service. We should do the same here.
Actions