Skip to content

Prod deploy for new talent report tab#108

Merged
jmgasper merged 6 commits into
masterfrom
develop
Jun 16, 2026
Merged

Prod deploy for new talent report tab#108
jmgasper merged 6 commits into
masterfrom
develop

Conversation

@jmgasper

Copy link
Copy Markdown
Collaborator

No description provided.

vas3a and others added 6 commits June 14, 2026 23:12
What was broken
Reports API had no endpoint to summarize open-to-work members by preferred role or export contact data for administrators.

Root cause
The openToWork preferredRoles trait was not exposed through reports-api; existing member search used only the available-for-gigs flag and skill search filters.

What was changed
Added admin-only open-to-work Talent JSON and CSV endpoints, query DTOs, SQL aggregation and export logic, Wipro email exclusion, sorting by wins and rating, and contact export fields.

Any added/updated tests
Added service, controller, and guard tests for the new endpoints and export. Verified with pnpm test -- member-search member-talent-report, pnpm lint, and pnpm build. The full pnpm test suite was run and still has unrelated pre-existing SFDC and report-directory failures.
PM-5349: add open-to-work talent report API
PM-5073 - Allow search by preferred roles, remove restriction about skills
What was broken
QA reported that Talent Manager role users still could not access the open-to-work Talent report.

Root cause
The previous API fix guarded the Talent JSON and CSV endpoints for administrators only, even though the follow-up requirement allows Talent Manager users too.

What was changed
Updated the Talent report guard to allow Talent Manager users in addition to administrators and all-reports machine clients. Updated endpoint documentation and Swagger 403 descriptions to match the expanded role access.

Any added/updated tests
Updated MemberTalentReportGuard tests to allow Talent Manager roles, including the Topcoder-prefixed role claim.
PM-5349: allow Talent Managers to access Talent report API
@jmgasper jmgasper merged commit b6124cd into master Jun 16, 2026
5 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.

2 participants