-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature coauthor support #102
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for kixlab ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
) | ||
)} | ||
</AuthorList> | ||
const printAuthor = (author: Member | string, asterisks: string) => { |
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.
Since this returns a component, the name should be PrintAuthor
. I would change the name to Name
@@ -164,6 +164,7 @@ export const MEMBERS: Record<string, Member> = { | |||
email: '[email protected]', | |||
kixlabPosition: 'Postdoc Researcher', | |||
img: 'daehyun.jpg', | |||
isAlumni: true, |
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.
why can't we use endDate to indicate if the person is an alumnus? I would change endYear to endDate
startSeason: 'Summer', | ||
}, | ||
// How should we handle Sung-Chul's entry? | ||
// sungchullee: { |
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.
is this discussed with Juho?
this branch is checked out from the other branch. If you are merging a branch into main, the branch should be from main. |
Closes #62