Bug/74521 work package on closed sprint cannot be edited#22986
Open
ulferts wants to merge 2 commits intorelease/17.4from
Open
Bug/74521 work package on closed sprint cannot be edited#22986ulferts wants to merge 2 commits intorelease/17.4from
ulferts wants to merge 2 commits intorelease/17.4from
Conversation
ulferts
force-pushed
the
bug/74521-work-package-on-closed-sprint-cannot-be-edited
branch
from
d61ccdc to
e322e8d
Compare
By extracting the assignable scope from the contract, the setup becomes simpler. This then lead to merging two validations into one. It also contains a drive by fix in the sprints api where only non completed sprints would be visible.
ulferts
force-pushed
the
bug/74521-work-package-on-closed-sprint-cannot-be-edited
branch
from
e322e8d to
14b9d26
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ticket
https://community.openproject.org/wp/74521
What are you trying to accomplish?
The PR extracts the assignability of sprints into a scope.
Merge checklist