Add /events page #159

Merged
j285he merged 45 commits from feat/events-page into main 2 years ago
j285he commented 2 years ago
Collaborator

Closes #111

Blocked by !158

Closes #111 Blocked by !158
j285he added 38 commits 2 years ago
b733acd156 Add lib file
bfcbf5c687 Fix type error
9f7d40b426 Merge branch 'main' into feat/events-lib
c39b0036ad Correct Dirents
590b2554c0 Fixes for dirents
d237621d14 ESLint fixes
3a697c09c0 Add sorting
b32fd53d95 Remove unneeded dirent conversion
40d37c4aea Fix lint problems
fecd3bf3e0 Rename functions
2798efc221 Function renaming
5dd258ec86 Merge branch 'main' into feat/events-year
408275e8cf Remove temp page
10449e921e Remove events
6f429d20bf Add events
29f2c3208c Fix build error
50c2f08769 Simplify types
a23a10d8e4 Fix for Fall 2018
416e027ebb Add futureEvents pastEvents EventCard logic
e971c85afc Create events/index.tsx
5631217681 Add time detection for current time
3e61407643 Remove events
03197ee048 Add index.tsx
2b88ea2f25 Remove events
j285he added the
Blocked
label 2 years ago
j285he requested review from a3thakra 2 years ago
j285he added 2 commits 2 years ago
j285he reviewed 2 years ago
import { GetStaticProps } from "next";
import Term, {
getStaticProps as termGetStaticProps,
Poster
Collaborator

import getCurrentTerm, getProps

import getCurrentTerm, getProps
j285he marked this conversation as resolved
a3thakra removed the
Blocked
label 2 years ago
a3thakra added 1 commit 2 years ago
j285he added 3 commits 2 years ago
Poster
Collaborator

Small issue -- when on /events, clicking the current term (e.g. Spring 2021) directs you to /events/2021/spring and so the shapes in the background change.

It's not a big deal but it's a little strange. Should this be an issue?

Small issue -- when on `/events`, clicking the current term (e.g. Spring 2021) directs you to `/events/2021/spring` and so the shapes in the background change. It's not a big deal but it's a little strange. Should this be an issue?
a3thakra added 1 commit 2 years ago
7ceffc3ba0 PR feedback
Collaborator

It's not a big deal but it's a little strange. Should this be an issue?

It's fine!

> It's not a big deal but it's a little strange. Should this be an issue? It's fine!
a3thakra approved these changes 2 years ago
j285he merged commit 5933dd03b8 into main 2 years ago
j285he referenced this issue from a commit 2 years ago
j285he deleted branch feat/events-page 2 years ago

Reviewers

a3thakra approved these changes 2 years ago
continuous-integration/drone/push Build is passing
The pull request has been merged as 5933dd03b8.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: www/www-new#159
Loading…
There is no content yet.