-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Added Certificates and Publications section. #26
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Added Certificates and publications section in CV #17 . |
certifications and publications
Added ResumeData Object for better readability and ease of modification. |
Added optional grade field in education #32 |
src/data/resume-data.tsx
Outdated
@@ -18,8 +18,71 @@ import { | |||
YearProgressLogo, | |||
} from "@/images/logos"; | |||
import { GitHubIcon, LinkedInIcon, XIcon } from "@/components/icons"; | |||
import { StaticImageData } from "next/image"; | |||
type ResumeData = { |
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.
I think the resume data should not have a type otherwise is going to cause a lot of problems for people who just want to fork the project and use it. some users may not have a project, websiteurl or don't want to put out location links etc.
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.
Yup, Fair point
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.
looks good now!!!!
PR should not contain changes to resume-data, it's my cv after all ;) |
No description provided.