Skip to content

Commit 30839b6

Browse files
authored
qfix: ensure target mappings are not undefined (#7936)
Signed-off-by: Alexander Onnikov <[email protected]>
1 parent 62dfae9 commit 30839b6

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

Diff for: services/github/pod-github/src/sync/issueBase.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ export abstract class IssueSyncManagerBase {
214214
const target: IssueSyncTarget | undefined =
215215
milestone !== undefined
216216
? {
217-
mappings: milestone.mappings,
217+
mappings: milestone.mappings ?? [],
218218
project: prj,
219219
target: milestone
220220
}
@@ -1161,7 +1161,7 @@ export abstract class IssueSyncManagerBase {
11611161
}
11621162
return {
11631163
project,
1164-
mappings: milestone.mappings,
1164+
mappings: milestone.mappings ?? [],
11651165
target: milestone,
11661166
prjData: external.projectItems.nodes.find((it) => it.project.id === milestone.projectNodeId)
11671167
}
@@ -1172,7 +1172,7 @@ export abstract class IssueSyncManagerBase {
11721172
getProjectIssueTarget (project: GithubProject, external?: IssueExternalData): IssueSyncTarget {
11731173
return {
11741174
project,
1175-
mappings: project.mappings,
1175+
mappings: project.mappings ?? [],
11761176
target: project,
11771177
prjData: external?.projectItems.nodes.find((it) => it.project.id === project.projectNodeId)
11781178
}

Diff for: services/github/pod-github/src/sync/projects.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -608,7 +608,7 @@ export class ProjectsSyncManager implements DocSyncManager {
608608
okit: Octokit
609609
): Promise<{ projectStructure: GithubProjectV2, wasUpdates: boolean, mappings: GithubFieldMapping[] }> {
610610
let projectStructure = await this.queryProjectStructure(integration, target)
611-
let mappings = target.mappings
611+
let mappings = target.mappings ?? []
612612

613613
if (projectStructure === undefined) {
614614
if (this.client.getHierarchy().isDerived(tracker.class.Project, target._class)) {
@@ -651,7 +651,7 @@ export class ProjectsSyncManager implements DocSyncManager {
651651
const mHash = JSON.stringify(mappings)
652652
// Create any platform field into matching github field
653653
for (const [, f] of allFields.entries()) {
654-
const existingField = (mappings ?? []).find((it) => it._id === f._id)
654+
const existingField = mappings.find((it) => it._id === f._id)
655655
if (f.hidden === true) {
656656
continue
657657
}

0 commit comments

Comments
 (0)