Skip to content

feat: centralize event status logic and add status badges to EventCard#101

Open
hasansyed107 wants to merge 11 commits into
data-umbrella:mainfrom
hasansyed107:feat/event-status-badges
Open

feat: centralize event status logic and add status badges to EventCard#101
hasansyed107 wants to merge 11 commits into
data-umbrella:mainfrom
hasansyed107:feat/event-status-badges

Conversation

@hasansyed107
Copy link
Copy Markdown

@hasansyed107 hasansyed107 commented Mar 28, 2026

Overview

This PR centralizes event status logic and adds clear status badges to each event card.

What’s included

  • Created a reusable getEventStatus helper in utils/eventHelpers.js
  • Handles:
    • Upcoming events
    • Past events
    • Today events
    • Invalid or missing dates safely
  • Integrated status display into EventCard
  • Added a simple visual indicator (dot + label) for better clarity

Why this matters

  • Removes duplicated date logic across components
  • Improves UI by clearly showing event relevance
  • Creates a foundation for future features like:
    • Sorting (upcoming first)
    • Filtering by status
    • Live event support

Notes

  • Kept UI minimal and consistent with existing design
  • Focused on clean, reusable logic

Closes #100

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 28, 2026 •
edited
Loading

❌ Deploy Preview for dueventboard failed. Why did it fail? →

Name Link
🔨 Latest commit 779cfcc
🔍 Latest deploy log https://app.netlify.com/projects/dueventboard/deploys/69d7527cb1b08e00086293ae

@hasansyed107
Copy link
Copy Markdown
Author

hasansyed107 commented Mar 28, 2026

image

@hasansyed107
Copy link
Copy Markdown
Author

hasansyed107 commented Mar 30, 2026

Hi @yuvimittal @xmnlab,

All CI checks are passing and the branch is fully rebased on main. This PR is ready for review whenever you get time.

Thanks!

@hasansyed107
Copy link
Copy Markdown
Author

hasansyed107 commented Mar 30, 2026

Hi @yuvimittal @xmnlab 👋

This PR introduces a dynamic Event Detail Page and extends the app from a list view into a more complete event exploration experience.

Highlights

  • Added routing support for /event/:id
  • Made EventCard clickable for deep linking
  • Implemented a detailed EventDetail page with status, capacity, and metadata
  • Maintained UI consistency with existing components
  • Added subtle entry animation for improved UX

All existing tests are passing, and the changes integrate cleanly with the current structure.

I’d appreciate your feedback when you have time — happy to iterate if needed!

sanvishukla and others added 8 commits April 4, 2026 08:22
@sanvishukla @hasansyed107
feat: Improve the event geocoding pipeline to reduce API calls (data-…
b3bd1e4 
…umbrella#87)

* feat: optimize geocoding with surgical YAML updates and visible jitter

* chore: remove temporary coordinate jittering
@Isha-Lama @hasansyed107
feat: add dynamic event status badges (data-umbrella#86)
61f29cd
@sanvishukla @hasansyed107
feat: Add about section at the end of page (data-umbrella#52)
4d2bde3 
* feat: add customized and responsive data umbrella footer

* feat: update footer component and styles

* correct prettier

* fix: footer visibility in light mode and internal navigation

* fix: update logo to colorful version and set default navigation links

* fix: update logo to colorful version and set default navigation links

* fix: update logo to colorful version and set default navigation links
@sanvishukla @hasansyed107
feat: improve map view with automatic geolocation and fit-bounds logic (
d4223ad 
data-umbrella#89)
@nidhipawar5 @hasansyed107
fix: remove hardcoded count in test file to prevent failures on addit…
102c482 
…ion of new events (data-umbrella#82)

* branch rebase

* fix

* package-lock fix

---------

Co-authored-by: Nidhi Pawar <nidhipawar@Nidhis-MacBook-Air.local>
@sanvishukla @hasansyed107
fix: prevent start date from being after end date in custom range (da…
bd1f943 
…ta-umbrella#112)
@hasansyed107
style: apply pre-commit auto fixes
0da78e6
@sanvishukla @hasansyed107
feat: add horizontal List view parallel to Grid view (data-umbrella#113)
29d0e39 
* feat: add horizontal List view parallel to Grid view

* fix: resolve file conflicts and format layout

* change UI

* Fix pre-commit issues
@yuvimittal
Copy link
Copy Markdown
Contributor

yuvimittal commented Apr 8, 2026

@hasansyed107 , the CI/CD seems to be failing

@hasansyed107 hasansyed107 force-pushed the feat/event-status-badges branch from 3a6a7e5 to 29d0e39 Compare April 9, 2026 06:52
@hasansyed107 hasansyed107 mentioned this pull request Apr 9, 2026
Open
@hasansyed107
Copy link
Copy Markdown
Author

hasansyed107 commented Apr 9, 2026

@yuvimittal
Hi
Since working on this, I continued development in #160, which now includes the same functionality along with additional fixes and improvements.

This PR has fallen out of sync and is showing conflicts, while #160 is already rebased on the latest main and passing all checks.

It might be best to continue with #160 instead. Please let me know your thoughts!

@sanvishukla
Copy link
Copy Markdown
Contributor

sanvishukla commented May 14, 2026

@yuvimittal, we have a feature showing upcoming and ended.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[ENHANCEMENT] Centralize Event Status Logic and Add Status Badges to EventCard

5 participants

@hasansyed107 @yuvimittal @sanvishukla @Isha-Lama @nidhipawar5