Conversation
aedwardg
left a comment
There was a problem hiding this comment.
Other than where the Staff type is located, this looks good code-wise.
My only hesitation is whether this is the direction we want to go with the About Page.
In the past (under the CC name) we tried to do a similar "image & bio" approach for server staff and it was very difficult to get the staff to open PRs with their info (or provide it to us so we could open the PR).
That said, I'm open to trying it again.
We could maybe start with only Admins and then ping the rest of the staff and tell them if they want to be included they need to open a PR.
src/types.d.ts
Outdated
There was a problem hiding this comment.
This should probably just go in the StaffCard component file.
.d.ts files are typically reserved for type declarations from libraries and you almost never need to create them yourself, as most libraries have types on the DefinitelyTyped repo.
You'd only create your own if DefinitelyTyped doesn't have the types you need for a library you're using.
See Typescript handbook here
There was a problem hiding this comment.
So in this instance, would you like to leave the .d.ts file? I'll migrate the Staff type to the StaffCard component either way.
There was a problem hiding this comment.
No, let's get rid of the .d.ts file, we shouldn't need one for this project.
aedwardg
left a comment
There was a problem hiding this comment.
Let's go with this and if we wanna change it up later we can. Nice work!
What issue is this solving?
Closes #39
Description
I went ahead and created the About page. It will feature a brief summary of the community, and a card for any of us that wish to share a bit about ourselves. I wasn't sure where to start on the summary, so I left some placeholder text in the meantime.
Staff cards are created when we add our info to
staff-info.jsonwhich is located in the About directory. All staff info should adhere to the new Staff type which is defined in the new global type file. And any images are stored in the public directory.Any helpful knowledge/context for the reviewer?
The duplicates are just to demonstrate the spacing.
Feelings gif (optional)
Please make sure you've attempted to meet the following coding standards