-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
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 |
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.
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
paddingRight: "2rem", | ||
}), | ||
}} | ||
onChange={(option) => setClassificationFilter(option as Array<iOption>)} |
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.
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!
Co-Authored-By: Galen Winsor <[email protected]>
I will want to add the remaining pubs, but making a PR so that I can get comments on style!