Skip to content

Commit 300eb28

Browse files
authored
Remove check for hasChildren (#537)
The current iteration has the flag for "hasChildren" set to true, even though it has no children. But, because that flag was set, it was filtered out of the possible values for an iteration. This fix does mean some very old values are included in the list of "all iterations". Those don't match any of the current patterns, so I chose the smallest code change possible. This logic could be modified to remove iterations from old patterns, but that seems unnecessary at this time.
1 parent f0d3212 commit 300eb28

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

actions/sequester/Quest2GitHub/QuestGitHubService.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,6 @@ private async Task<QuestIteration[]> RetrieveIterationLabelsAsync()
226226

227227
var parentIteration = sprintPackets.Descendent("value").EnumerateArray().Single(i => i.GetProperty("structureType").GetString() == "iteration");
228228

229-
230229
return [.. ChildIterations(parentIteration)];
231230

232231
static IEnumerable<QuestIteration> ChildIterations(JsonElement parentIteration)
@@ -240,7 +239,7 @@ static IEnumerable<QuestIteration> ChildIterations(JsonElement parentIteration)
240239
yield return child;
241240
}
242241
}
243-
else if (sprintElement.GetProperty("hasChildren").GetBoolean() == false)
242+
else
244243
{
245244
var iteration = ConstructIteration(sprintElement);
246245
if (iteration is not null)

0 commit comments

Comments
 (0)