Skip to content
This repository has been archived by the owner on Dec 19, 2024. It is now read-only.

feat: init pubs page #55

Merged
merged 30 commits into from
Jul 2, 2024
Merged

feat: init pubs page #55

merged 30 commits into from
Jul 2, 2024

Conversation

hetd54
Copy link
Collaborator

@hetd54 hetd54 commented Jun 27, 2024

I will want to add the remaining pubs, but making a PR so that I can get comments on style!

@hetd54 hetd54 requested review from broarr and galenwinsor June 27, 2024 15:11
Copy link

github-actions bot commented Jun 27, 2024

Visit the preview URL for this PR (updated for commit 464be96):

https://mmp-site-b1c9b--pr55-pubs-page-wi1p564n.web.app

(expires Mon, 08 Jul 2024 20:53:53 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 4eb870c89e876f1812e204af417359065d2a23b1

Copy link
Contributor

@galenwinsor galenwinsor left a comment

Choose a reason for hiding this comment

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

Awesome! Just a minor comment to clean things up. Then I'm thinking that I can tackle styling so you don't have to worry about that, if that works for you.

src/components/Publications.tsx Outdated Show resolved Hide resolved
src/components/Publications.tsx Outdated Show resolved Hide resolved
src/components/Publications.tsx Show resolved Hide resolved
@hetd54 hetd54 requested a review from galenwinsor July 1, 2024 17:52
paddingRight: "2rem",
}),
}}
onChange={(option) => setClassificationFilter(option as Array<iOption>)}
Copy link
Contributor

Choose a reason for hiding this comment

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

To make this work without the cast, you need to add an as const after the initial classifications value and type the state variable explicitly rather than implicitly. You'll also need to mark that type definition with Readonly<>. Let me know if that doesn't make sense!

@hetd54 hetd54 requested a review from galenwinsor July 2, 2024 16:59
@galenwinsor galenwinsor merged commit 7c4e807 into main Jul 2, 2024
2 checks passed
@galenwinsor galenwinsor deleted the pubs-page branch July 2, 2024 19:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants