Skip to content

Commit 71e2328

Browse files
committed
Only sync progress reports on project update when there is a date change
Previously every project update was listing engagements when we knew ahead of time that the reports were in sync because there were no date changes
1 parent 2fba5fb commit 71e2328

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

src/components/periodic-report/handlers/sync-progress-report-to-engagement.handler.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,15 @@ export class SyncProgressReportToEngagementDateRange
5050
return;
5151
}
5252

53+
if (
54+
event instanceof ProjectUpdatedEvent &&
55+
event.changes.mouStart === undefined &&
56+
event.changes.mouEnd === undefined
57+
) {
58+
// Project dates haven't changed, so do nothing.
59+
return;
60+
}
61+
5362
if (
5463
(event instanceof EngagementCreatedEvent && event.engagement.changeset) ||
5564
(event instanceof EngagementUpdatedEvent && event.updated.changeset)

0 commit comments

Comments
 (0)