Skip to content

Conversation

@codeunia-dev
Copy link
Owner

@codeunia-dev codeunia-dev commented Nov 19, 2025

This refactor improves consistency across upcoming event and hackathon registration data handling, while simplifying the component structure.

🔄 Changes Included

  • Standardized the data format for upcoming event and hackathon registrations to ensure consistent usage across the app.

  • Updated UpcomingHackathonItem:

    • Replaced the wrapper element with a <div> for improved flexibility and DOM consistency.
    • Removed the unused companySlug prop to reduce noise and prevent confusion.
  • Cleaned up minor unused logic for better maintainability.

🎯 Why This Matters

  • Reduces inconsistencies in how event/hackathon data flows through the UI.
  • Makes the component easier to maintain and extend.
  • Helps future features relying on unified registration models.

🗂️ Commit Summary

  • 1 file changed
  • 1 commit

Authored by: @akshay0611

Summary by CodeRabbit

  • Style
    • Changed hackathon registration label from "teams registered" to "participants" for improved clarity.
    • Removed clickable navigation from hackathon items in the dashboard display.

…update `UpcomingHackathonItem` to a div, removing unused `companySlug` prop.
@vercel
Copy link

vercel bot commented Nov 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
codeunia Building Building Preview Comment Nov 19, 2025 10:55am

@codeunia-dev codeunia-dev merged commit f797ec9 into main Nov 19, 2025
2 of 4 checks passed
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 19, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

The PR removes the companySlug prop from UpcomingHackathonItem, eliminates its navigation link wrapper, normalizes registration count data for events and hackathons, and relabels the hackathon registration text from "teams registered" to "participants".

Changes

Cohort / File(s) Summary
Dashboard component updates
components/dashboard/CompanyDashboard.tsx
Removed companySlug prop from UpcomingHackathonItemProps interface and function signature; removed navigation link wrapper around UpcomingHackathonItem; normalized registration counts using || 0 for both events and hackathons; updated UI label from "teams registered" to "participants"; removed companySlug from component usage.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Single file with focused, cohesive changes
  • Straightforward prop removal and data normalization
  • Clear UI text update with no complex logic

Possibly related PRs

Poem

🐰 A prop takes flight, the link unwrapped with care,
Registration counts now normalized fair,
"Participants" sing where teams once cheered,
Dashboard simplified, the path has cleared! 🎉

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/analyticsgraph

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e3fffc5 and 23a8619.

📒 Files selected for processing (1)
  • components/dashboard/CompanyDashboard.tsx (5 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

3 participants