Add [event].tsx page #153

Merged
a258wang merged 7 commits from feat/event-info-page into main 2 years ago
Owner

This is the [event].tsx page that displays a single event, with a couple of sample events in content/events for testing purposes.

Closes #114

This is the `[event].tsx` page that displays a single event, with a couple of sample events in `content/events` for testing purposes. Closes #114
a258wang added 4 commits 2 years ago
a258wang changed title from [event].tsx page to Add [event].tsx page 2 years ago
a3thakra reviewed 2 years ago
await Promise.all(
years.map(async (year) => {
const termsInYear = await getEventTermsByYear(year);
return termsInYear.map((term) => ({ year, term }));
Collaborator

You already have access to the terms here, might as well get all the events of that term here! That will result in a 3d array, but we don't really care because .flat() will convert that to a 1d array.

You already have access to the terms here, might as well get all the events of that term here! That will result in a 3d array, but we don't really care because `.flat()` will convert that to a 1d array.
a3thakra approved these changes 2 years ago
a3thakra left a comment
Collaborator

Just 1 nit-pick, LGTM otherwise!

Make sure to delete the markdown files before merging!!

Just 1 nit-pick, LGTM otherwise! Make sure to delete the markdown files before merging!!
a258wang added 1 commit 2 years ago
0b62d2810e Streamline event info page getStaticPaths
a258wang force-pushed feat/event-info-page from 2b186fc162 to 0b62d2810e 2 years ago
a258wang added 1 commit 2 years ago
a258wang added 1 commit 2 years ago
9881caf828 Merge branch 'main' into feat/event-info-page
a258wang merged commit 6f24b266eb into main 2 years ago
a258wang referenced this issue from a commit 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 6f24b266eb.
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#153
Loading…
There is no content yet.