Skip to content

Show events on home - #31

Open
eth-p wants to merge 6 commits into
mainfrom
show-events-on-home
Open

eth-p wants to merge 6 commits into
mainfrom
show-events-on-home

Conversation

@eth-p

@eth-p eth-p commented Jul 19, 2022

Copy link
Copy Markdown
Collaborator

This PR makes it possible to automatically show upcoming events on the home page.

@netlify

netlify Bot commented Jul 19, 2022

Copy link
Copy Markdown

Deploy Preview for jolly-feynman-7082d6 ready!

Name Link
🔨 Latest commit 4f0db74
🔍 Latest deploy log https://app.netlify.com/sites/jolly-feynman-7082d6/deploys/62e0ba8d7e8510000921165e
😎 Deploy Preview https://deploy-preview-31--jolly-feynman-7082d6.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@rubik516

Copy link
Copy Markdown
Collaborator

I consulted Nathan last time when I updated the Events page, and I believe we want the event sticky notes to be displayed in 3 columns (similar to the current Events page), so I also made some styling change for the Events page. Maybe it's worth double checking with Design team again for the Home page?

@eth-p

eth-p commented Jul 19, 2022

Copy link
Copy Markdown
Collaborator Author

Thanks for the heads up on that! I'll bring it up during today's meeting

@eth-p

eth-p commented Jul 27, 2022

Copy link
Copy Markdown
Collaborator Author

Just asked Nathan, and he said to update it to a three-column layout. I'll rebase, fix merge conflicts, and do that. Will let you know when done!

eth-p added 2 commits July 26, 2022 20:25
This array is automatically sorted to have the closest events
(chronologically) near the beginning of the array.
@eth-p
eth-p force-pushed the show-events-on-home branch from d971cf1 to 67a072c Compare July 27, 2022 03:29
@eth-p
eth-p requested a review from rubik516 July 27, 2022 04:13
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.

2 participants