From 62f7c071bfa4b0e2447a109235fb47460f68603f Mon Sep 17 00:00:00 2001 From: yu-i-i Date: Thu, 12 Jun 2025 19:14:36 +0200 Subject: [PATCH] Track changes: use getTrackedChangesUserIds introduced in the mainstream --- .../track-changes/app/src/TrackChangesController.js | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/services/web/modules/track-changes/app/src/TrackChangesController.js b/services/web/modules/track-changes/app/src/TrackChangesController.js index 45f8a03e0f..e66a49d3d4 100644 --- a/services/web/modules/track-changes/app/src/TrackChangesController.js +++ b/services/web/modules/track-changes/app/src/TrackChangesController.js @@ -59,16 +59,10 @@ const TrackChangesController = { // provided here to assign names to authors who have left the project but still have unaccepted changes. try { const { project_id } = req.params - const memberIds = new Set() - const ranges = await DocstoreManager.promises.getAllRanges(project_id) - ranges.forEach(range => { - ;[...range.ranges?.changes || [], ...range.ranges?.comments || []].forEach(item => { - memberIds.add(item.metadata?.user_id) - }) - }) + const memberIds = await DocstoreManager.promises.getTrackedChangesUserIds(project_id) const limit = pLimit(3) const users = await Promise.all( - [...memberIds].map(memberId => + memberIds.map(memberId => limit(async () => { const user = await UserInfoManager.promises.getPersonalInfo(memberId) return UserInfoController.formatPersonalInfo(user)