Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add snippet of events page to home page #76

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

Conversation

areebakq
Copy link
Contributor

Created components/eventList.js to store events array (this is the one that will be routinely updated). Rendered eventsList array to pages/events.js and snippet version to pages/home.js.

Copy link
Contributor

@Aarushi-Pandey Aarushi-Pandey left a comment

Choose a reason for hiding this comment

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

Everything looks great! I was wondering whether flyers would also be visible for events in the home page

}

export const rows = [
createData(upcomingEvents[0], flyers[0].image),
Copy link
Member

Choose a reason for hiding this comment

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

it would prob be easier to just have one array of objects rather than making sure the indexes of each match

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

Successfully merging this pull request may close these issues.

3 participants