Skip to content

Commit

Permalink
combine CurrentEmployees and PastEmployees under one function
Browse files Browse the repository at this point in the history
  • Loading branch information
tomas-salgado committed Jul 26, 2024
1 parent 6b769bb commit ae4e6f2
Showing 1 changed file with 33 additions and 45 deletions.
78 changes: 33 additions & 45 deletions apps/member-profile/app/routes/_profile.companies_.$id.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -194,12 +194,7 @@ export default function CompanyPage() {
</ul>
</nav>
<Divider my="4" />
{activeTab === 'employees' && (
<>
<CurrentEmployees />
<PastEmployees />
</>
)}
{activeTab === 'employees' && <Employees />}
{activeTab === 'reviews' && <ReviewsList />}
{activeTab === 'interview-reviews' && <InterviewReviewsList />}
</section>
Expand Down Expand Up @@ -360,47 +355,40 @@ function InterviewReviewsList() {
);
}

function CurrentEmployees() {
const { currentEmployees } = useLoaderData<typeof loader>();
function Employees() {
const { currentEmployees, pastEmployees } = useLoaderData<typeof loader>();

return (
<Card>
<Card.Title>Current Employees ({currentEmployees.length})</Card.Title>

{currentEmployees.length ? (
<ul>
{currentEmployees.map((employee) => {
return <EmployeeItem key={employee.id} employee={employee} />;
})}
</ul>
) : (
<Text color="gray-500">
There are no current employees from ColorStack.
</Text>
)}
</Card>
);
}

function PastEmployees() {
const { pastEmployees } = useLoaderData<typeof loader>();

return (
<Card>
<Card.Title>Past Employees ({pastEmployees.length})</Card.Title>

{pastEmployees.length ? (
<ul>
{pastEmployees.map((employee) => {
return <EmployeeItem key={employee.id} employee={employee} />;
})}
</ul>
) : (
<Text color="gray-500">
There are no past employees from ColorStack.
</Text>
)}
</Card>
<>
<Card>
<Card.Title>Current Employees ({currentEmployees.length})</Card.Title>
{currentEmployees.length ? (
<ul>
{currentEmployees.map((employee) => {
return <EmployeeItem key={employee.id} employee={employee} />;
})}
</ul>
) : (
<Text color="gray-500">
There are no current employees from ColorStack.
</Text>
)}
</Card>
<Card>
<Card.Title>Past Employees ({pastEmployees.length})</Card.Title>
{pastEmployees.length ? (
<ul>
{pastEmployees.map((employee) => {
return <EmployeeItem key={employee.id} employee={employee} />;
})}
</ul>
) : (
<Text color="gray-500">
There are no past employees from ColorStack.
</Text>
)}
</Card>
</>
);
}

Expand Down

0 comments on commit ae4e6f2

Please sign in to comment.