Skip to content

Commit 7e4b0c3

Browse files
deocagunotclaude
andcommitted
fix(codevs): show accepted developers on Be a Member
The Be a Member page filtered developers on role_id 10, but the accept flow sets role_id 4 (Intern). Accepted developers therefore never appeared there, and whoever still held the legacy role_id 10 showed up instead — the "iba ung devs rendered" report. application_status is the reliable signal: the accept flow always sets it to "passed", while role_id tracks the Intern → Codev → Mentor progression and changes as a developer advances. Filtering on it matches what /profiles already does, so both public pages now read from the same source of truth. The existing post-filter is unchanged and still removes admins, failed and applying codevs, then narrows to active — so no one who should be hidden becomes visible. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent aeea675 commit 7e4b0c3

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

‎apps/codebility/app/(marketing)/codevs/_components/CodevsProfiles.tsx‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,11 @@ export const revalidate = 0;
1515

1616
export default async function CodevsProfiles() {
1717
const [{ data: allCodevs, error }] = await Promise.all([
18-
getCodevs({ filters: { role_id: 10 } }), // show only Codevs with role_id 10 which is Codev
18+
// Accepted developers are identified by application_status, not by role_id.
19+
// The accept flow sets application_status "passed" while role_id tracks the
20+
// Intern → Codev → Mentor progression, so filtering on role_id 10 hid every
21+
// newly accepted developer. /profiles already uses this same filter.
22+
getCodevs({ filters: { application_status: "passed" } }),
1923
]);
2024

2125
if (error) {

0 commit comments

Comments
 (0)