Skip to content
This repository has been archived by the owner on Jan 13, 2019. It is now read-only.

Past Events page #59

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Past Events page #59

wants to merge 4 commits into from

Conversation

XiayangFan
Copy link
Contributor

No description provided.

@@ -0,0 +1,2 @@
taeklklej
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wat

@@ -0,0 +1,140 @@
/* fuck */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

? why is there this and events.css?

@@ -45,6 +45,7 @@
<a href="#">Student Services</a>
<ul id="subnav-2" class="subnav">
<li><a href="/oh">Office Hours</a></li>
<li><a href="/events">Past Events</a></li>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think it would be better to have this url be pastevents or past_events to avoid confusion

@Baisang
Copy link
Contributor

Baisang commented Dec 6, 2016

will test locally over break, this commit is contingent on our facebook scraper running again (right now it's turned off)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants