Skip to content

Commit c38bc14

Browse files
committed
feat: sort arrays in CD materials approval info and user group data
1 parent ee3b496 commit c38bc14

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

src/Common/Common.service.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -206,22 +206,22 @@ const processCDMaterialsApprovalInfo = (enableApproval: boolean, cdMaterialsResu
206206
acc[email] = {
207207
email,
208208
hasAccess: index % 2 === 0,
209-
hasApproved: index % 2 !== 0,
210209
}
211210
return acc
212211
}, {} as Record<string, UserGroupApproverType>),
213212
requiredCount: SPECIFIC_EMAILS.length,
213+
// TODO: Sort these arrays
214214
emails: SPECIFIC_EMAILS,
215215
},
216216
userGroupData: VALID_GROUPS.reduce((acc, userGroup) => {
217+
// TODO: Sort these arrays
217218
const emails = [`${userGroup}-1@devtron.ai`, `${userGroup}-2@devtron.ai`, `${userGroup}-3@devtron.ai`]
218219

219220
acc[userGroup] = {
220221
dataStore: emails.reduce((acc, email, index) => {
221222
acc[email] = {
222223
email,
223224
hasAccess: index % 2 === 0,
224-
hasApproved: index % 2 !== 0,
225225
}
226226
return acc
227227
}, {} as Record<string, UserGroupApproverType>),
@@ -230,6 +230,7 @@ const processCDMaterialsApprovalInfo = (enableApproval: boolean, cdMaterialsResu
230230
}
231231
return acc
232232
}, {} as Record<string, UserGroupDataType>),
233+
// TODO: Sort these arrays
233234
validGroups: VALID_GROUPS,
234235
}
235236
}

src/Common/Types.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -325,8 +325,6 @@ export interface UserApprovalConfigType {
325325

326326
export interface UserGroupApproverType {
327327
email: string
328-
// Will derive from approvedUsersData list
329-
hasApproved: boolean
330328
hasAccess: boolean
331329
}
332330

0 commit comments

Comments
 (0)