search

loolabs / waterpark

Waterloo's social park
http://waterpark.loo-labs.vercel.app
ISC License
4 stars 0 forks source link

💻 Add header to events page #137

Closed xujustinj closed 3 years ago

xujustinj commented 3 years ago

Screenshots

https://youtu.be/NKigUdlXkpg

Purpose

Closes #121

Also hook up event search to the events page. At the moment this performs filtering before everything else (including the creation of the map from days to events). This may or may not be ideal, pending future discussion.

Approach

Reuses the same header components from the clubs page, but duplicates the code. If we were to properly turn the header into a common component, the refactor could break a bunch of changes in the pipeline. Some future refactor is needed.

Testing

Visually verified in screencap

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/loo-labs/waterpark/J3PrJ2M3ZNKh75hRVtudDRH8m6ZH
✅ Preview: https://waterpark-git-events-header-loo-labs.vercel.app

xujustinj commented 3 years ago

Discussed with @wang-em, it makes sense to merge this now so that we have a header to use for reference when we do the calendar fixes for #134