Skip to content

85 create dashboardlistitem component - #208

Open
pebblefoot31 wants to merge 6 commits into
mainfrom
85-create-dashboardlistitem-component
Open

85 create dashboardlistitem component#208
pebblefoot31 wants to merge 6 commits into
mainfrom
85-create-dashboardlistitem-component

Conversation

@pebblefoot31

@pebblefoot31 pebblefoot31 commented Apr 19, 2024

Copy link
Copy Markdown
Collaborator

Description

This component is designed to ease the process of the applicant decision-making process. It includes an applicants name, status, and a button that is designed to make different decisions based on the status of the applicant and when it was last updated, For example, depending on the applicant's status we may want to send a follow-up email or an interview request.

Relevant issue(s)

Questions

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have assigned reviewers to this PR

@pebblefoot31 pebblefoot31 linked an issue Apr 19, 2024 that may be closed by this pull request
<span>
{props.applicant.firstName} {props.applicant.lastName}
</span>
<Chip label={props.applicant.status} />

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add the size="small" prop, and add a left margin on this to give it a lil breathing room (sx={{ ml: 2 }})

@zaviermiller zaviermiller left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small nits

@efisk15
efisk15 self-requested a review October 21, 2024 11:59
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.

Create DashboardListItem component

2 participants