Skip to content

feat: Recruitment timeline component#15

Merged
alexespejo merged 3 commits intomainfrom
recruitment-timeline
Feb 21, 2026
Merged

feat: Recruitment timeline component#15
alexespejo merged 3 commits intomainfrom
recruitment-timeline

Conversation

@xgraceyan
Copy link
Contributor

image

@vercel
Copy link

vercel bot commented Feb 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ctc-website-25-26 Ready Ready Preview, Comment Feb 6, 2026 7:11pm
website Error Error Feb 6, 2026 7:11pm

title: string;
}

const recruitmentEvents: TimelineEvent[] = [
Copy link
Member

Choose a reason for hiding this comment

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

nit: there's a pattern in the project where constants are stored in a file I don't remember the name of so that future webmasters can just change those files

Copy link
Member

@alexespejo alexespejo left a comment

Choose a reason for hiding this comment

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

Looks good. Small nit

Copy link
Member

@alexespejo alexespejo left a comment

Choose a reason for hiding this comment

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

😛


export const RECRUITMENT_EVENTS: RecruitmentEvent[] = [
{
date: "September 21, 2025",
Copy link
Member

Choose a reason for hiding this comment

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

Nit: Might be overkill but maybe there's a way to create a utility that will make the dates dumb proof so all the dates can be configured in one variable or somn. Lwky dtm I just felt like commenting something :p

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ya idk exactly wym but i made the dates not strings so hopefully its easier to edit. a bit dtm tho i think

@xgraceyan xgraceyan requested a review from alexespejo February 6, 2026 19:11
@alexespejo alexespejo merged commit bc52a1d into main Feb 21, 2026
2 of 3 checks passed
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.

2 participants