Skip to content

Commit 5d9cf62

Browse files
authored
Leads can be the member in interview scheduler (#1082)
### Summary <!-- Required --> <!-- Provide a general summary of your changes in the Title above --> <!-- Itemize bug fixes, new features, and other changes --> <!-- Feel free to break this into sub-sections, i.e. features, fixes, etc. --> <!-- Some examples are shown below. --> This pull request is the first step towards implementing feature Foo - [x] implemented small changes so leads can choose the member slot in interview scheduler - [x] i tested it trust
1 parent 078b004 commit 5d9cf62

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

frontend/src/components/Interview-Scheduler/SchedulingSidePanel.tsx

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,10 @@ const SchedulingSidePanel: React.FC<{
5050

5151
const memberOptions = [
5252
{ text: 'Vacant' },
53-
...members
54-
.filter((mem) => !LEAD_ROLES.includes(mem.role))
55-
.map((mem) => ({
56-
text: `${mem.firstName} ${mem.lastName}`,
57-
value: mem.email
58-
}))
53+
...members.map((mem) => ({
54+
text: `${mem.firstName} ${mem.lastName}`,
55+
value: mem.email
56+
}))
5957
];
6058

6159
const applicantOptions = [

frontend/src/components/Interview-Scheduler/SlotHooks.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ export const useInterviewSlotStatus = (slot: InterviewSlot): SlotStatus => {
2828

2929
if (isLead) {
3030
if (slot.lead === null) return 'vacant';
31+
if (slot.members.some((mem) => mem && mem.email === userEmail)) return 'possessed';
3132
return slot.lead !== null && slot.lead.email === userEmail ? 'possessed' : 'occupied';
3233
}
3334
if (isMember) {

0 commit comments

Comments
 (0)