-
Notifications
You must be signed in to change notification settings - Fork 18
Update the People Page #32
Comments
@RohanSasne can this be assigned to me ! |
#33 |
Here's how we can modify the const contributors = [
// Existing contributors (with updated details of mentioned ones)
// New contributors
{
name: "Dongchen Zhang",
email: "[email protected]",
role: "PhD Student",
profession: "PhD Student in Dr. Michael Dietze's Lab, Boston University",
img: "image_link_here",
},
{
name: "Qianyu Li",
email: "[email protected]",
role: "Research Scientist",
profession: "Brookhaven National Lab",
img: "image_link_here",
},
// ... EOL
];
export default contributors; I will open a PR for the same. |
Great suggestion on adding an Alumni section! Here’s a brief outline of how we could implement it, or Rather how I would implement it. However, it may change based on requirements required OR as suggested by :
I'll draft this and share it for feedback before finalizing the PR. |
I have made a pull request regarding adding 2 people at people page. I want to make sure that do you need another section for the contributors i.e. new page and I am interested in contributing can open up a PR for the same issue |
We can improve the UI of the people page by using card animation using Figma. I can help you with this. |
Thanks for the suggestions @princyym . I guess you can open another PR after this to improve the UI but that is totally the decision of maintainers. |
Thanks for connecting and for the response, i will consider your suggestion
try to contribute more and more. Could you please suggest or assign me some
other issues it would be really helpful in this open-source journey, it
would be really helpfull.
Regards
Princy
…On Sat, Jan 20, 2024, 10:52 PM Abhinav Pandey ***@***.***> wrote:
Thanks for the suggestions @princyym <https://github.com/princyym> . I
guess you can open another PR after this to improve the UI but that is
totally the decision of maintainers.
—
Reply to this email directly, view it on GitHub
<#32 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BAWQJ24KDHLGGAZQJSJPZPDYPP4OJAVCNFSM6AAAAABBF3PMECVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBSGE4DCNZYGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Comments from @mdietze were:
People Page: https://pecanproject.github.io/web/people
Anyone interested in contributing can open up a PR for the same issue
The text was updated successfully, but these errors were encountered: