Skip to content

PM-6477 manager audit trail - #63

Merged
vas3a merged 4 commits into
developfrom
PM-6477_manager-audit-trail
Oct 1, 2026
Merged

vas3a merged 4 commits into
developfrom
PM-6477_manager-audit-trail

Conversation

@vas3a

@vas3a vas3a commented Sep 29, 2026

Copy link
Copy Markdown
Collaborator

This pull request introduces a new audit history endpoint for engagement manager assignments and improves timesheet entry visibility and filtering logic, especially for Talent Managers (TMs). It also updates tests to reflect the refined behavior and cleans up related service logic.

Engagement Manager Audit History:

  • Added a new GET /audit endpoint to EngagementManagersController that returns the assignment and removal history for engagement managers, using the TimesheetAuditRecordDto. This enables clients to view a detailed management audit trail. [1] [2]
  • Implemented the findAudit method in EngagementManagersService to fetch and format relevant audit records, ensuring proper permissions and engagement existence checks. [1] [2]

Timesheet Entry Visibility and Filtering:

  • Updated TimesheetsService so that TMs see only submitted entries by default, but can filter for approved entries if specified. This is reflected in both the service logic and the corresponding tests. [1] [2] [3]
  • Adjusted filtering logic in timesheet rollup queries to ensure TMs and administrators have appropriate access and filtering capabilities, removing redundant or overly restrictive conditions. [1] [2] [3]

@vas3a
vas3a merged commit 49ddad3 into develop Oct 1, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant