-
Notifications
You must be signed in to change notification settings - Fork 4
[INTW26] Implement Interview Scheduling Feature #263
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
chene0
wants to merge
13
commits into
staging
Choose a base branch
from
INTW26-interview-scheduling-feature
base: staging
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
1e49cd7
use 'fr' instead of 'px' for SplitPageLayout pane width values
chene0 d524cea
setup basic structure of interview group scheduling page and fetch
chene0 e66ae3e
fetch interview partner info on interview group scheduling page
chene0 4d6e929
implement interview group update submission flow (no ui feedback yet)
chene0 8da684c
add frontend feedback to calendly link submission flow
chene0 10ca932
complete left side panel design to match figma
chene0 7c37922
fix right side panel appearance
chene0 eb0a693
switch interview scheduler route to `/interview-groups?interviewGroup…
chene0 cf876a4
add gap between interview partner email and interview details
chene0 a6d2cff
add comment regarding auth based on currentAuthed user and users in the
chene0 3575005
rename InterviewGroupStatusEnum to InterviewGroupStatus
chene0 6f8cb20
remove router.query.interviewGroupId type assertion, move type narrowing
chene0 3604673
remove return type for edit icon component
chene0 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,71 @@ | ||
| import { fetchGraphql } from "@utils/makegqlrequest"; | ||
| import { queries, mutations } from "graphql/queries"; | ||
|
|
||
| const InterviewGroupStatus = { | ||
| READY_TO_INTERVIEW: "Ready to Interview", | ||
| INVITES_SENT: "Invites Sent", | ||
| AVAILABILITY_PENDING: "Availability Pending", | ||
| } as const; | ||
|
|
||
| type InterviewGroup = { | ||
| schedulingLink: string | null; | ||
| }; | ||
|
|
||
| type Applicant = { | ||
| firstName: string; | ||
| lastName: string; | ||
| }; | ||
|
|
||
| type Interviewer = { | ||
| id: string; | ||
| firstName: string; | ||
| lastName: string; | ||
| email: string; | ||
| profilePictureFileId: string | null; | ||
| }; | ||
|
|
||
| const InterviewGroupAPIClient = { | ||
| updateSchedulingLink: async ( | ||
| id: string, | ||
| schedulingLink: string, | ||
| ): Promise<string | null> => { | ||
| return fetchGraphql(mutations.updateInterviewGroup, { | ||
| id, | ||
| status: InterviewGroupStatus.READY_TO_INTERVIEW, | ||
| schedulingLink, | ||
| }) | ||
| .then((result) => result.data.updateInterviewGroup.schedulingLink) | ||
| .catch((e: Error) => { | ||
| throw new Error( | ||
| `Failed to update scheduling link. Cause: ${e.message}`, | ||
| ); | ||
| }); | ||
| }, | ||
| getInterviewGroup: async (id: string): Promise<InterviewGroup> => { | ||
| return fetchGraphql(queries.getInterviewGroup, { id }) | ||
| .then((result) => result.data.getInterviewGroup) | ||
| .catch((e: Error) => { | ||
| throw new Error(`Failed to fetch interview group. Cause: ${e.message}`); | ||
| }); | ||
| }, | ||
| getInterviewersByGroupId: async (groupId: string): Promise<Interviewer[]> => { | ||
| return fetchGraphql(queries.getInterviewersByGroupId, { groupId }) | ||
| .then((result) => result.data.getInterviewersByGroupId) | ||
| .catch((e: Error) => { | ||
| throw new Error(`Failed to fetch interviewers. Cause: ${e.message}`); | ||
| }); | ||
| }, | ||
| getInterviewedApplicantsByGroupId: async ( | ||
| groupId: string, | ||
| ): Promise<Applicant[]> => { | ||
| return fetchGraphql(queries.getInterviewedApplicantsByGroupId, { groupId }) | ||
| .then((result) => result.data.getInterviewedApplicantsByGroupId) | ||
| .catch((e: Error) => { | ||
| throw new Error( | ||
| `Failed to fetch interviewed applicants. Cause: ${e.message}`, | ||
| ); | ||
| }); | ||
| }, | ||
| }; | ||
|
|
||
| export default InterviewGroupAPIClient; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| export const EditIcon = () => ( | ||
| <svg | ||
| width="24" | ||
| height="24" | ||
| viewBox="0 0 24 24" | ||
| fill="none" | ||
| xmlns="http://www.w3.org/2000/svg" | ||
| > | ||
| <path | ||
| d="M11 4H4C3.46957 4 2.96086 4.21071 2.58579 4.58579C2.21071 4.96086 2 5.46957 2 6V20C2 20.5304 2.21071 21.0391 2.58579 21.4142C2.96086 21.7893 3.46957 22 4 22H18C18.5304 22 19.0391 21.7893 19.4142 21.4142C19.7893 21.0391 20 20.5304 20 20V13M18.5 2.5C18.8978 2.10217 19.4374 1.87868 20 1.87868C20.5626 1.87868 21.1022 2.10217 21.5 2.5C21.8978 2.89782 22.1213 3.43739 22.1213 4C22.1213 4.56261 21.8978 5.10217 21.5 5.5L12 15L8 16L9 12L18.5 2.5Z" | ||
| stroke="#848AA5" | ||
| strokeWidth="2" | ||
| strokeLinecap="round" | ||
| strokeLinejoin="round" | ||
| /> | ||
| </svg> | ||
| ); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
replaced
pxwithfrsuch that component scales horizontally to parentThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mxc-maggiechen what we thinking about this one, seems interesting