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

N°7348 - Enable to param maximum depth of impact analysis in function Ticket::UpdateImpactedItems #633

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

accognet
Copy link
Contributor

@accognet accognet commented Mar 13, 2024

In this kind of delta
image

we want to limit depth of impact analysis to 1 or 2. It's actually hard coded to 10.

@accognet accognet self-assigned this Mar 13, 2024
@accognet accognet added the internal Work made by Combodo label Mar 13, 2024
@Hipska
Copy link
Contributor

Hipska commented Mar 13, 2024

How and where would that be configured then?

@accognet
Copy link
Contributor Author

Directly in the delta on the function call

Co-authored-by: Thomas Casteleyn <thomas.casteleyn@super-visions.com>
@Molkobain
Copy link
Member

Accepted during support PRs review.
A migration check has been done, as this change will break existing overloads of this method.

During functional review, we will discuss whether it's ok or we should rather use a config param.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Work made by Combodo
Projects
Status: Pending functional review
3 participants