Skip to content

refactor: use listCard to render cfp categories #179

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mohitkh7
Copy link

@mohitkh7 mohitkh7 commented Aug 7, 2025

Issue: #168

Current implementation of cfp categories is HTML driven. This PR makes it Component driven by introducing 2 new components

  • list component
  • listCard component

Copy link

netlify bot commented Aug 7, 2025

Deploy Preview for inpycon2025 ready!

Name Link
🔨 Latest commit 21c6577
🔍 Latest deploy log https://app.netlify.com/projects/inpycon2025/deploys/6894de2f60144f00083a9105
😎 Deploy Preview https://deploy-preview-179--inpycon2025.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@mohitkh7 mohitkh7 force-pushed the refactor-flat-card branch from 4b6d9e3 to 21c6577 Compare August 7, 2025 17:11
@farhaanbukhsh
Copy link
Contributor

@mohitkh7 great work here, what I am thinking is if we can eliminate using flatCard and write a new component to be used here and slowly write new components replacing it through out the code base. Since that component is too specific and doesn't help in reducing the code. Let me know if you need more clarification.

@mohitkh7
Copy link
Author

@farhaanbukhsh I didn't fully understood it. Can you please elaborate it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants