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

Hide monthly events on contact page #647

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

casperboone
Copy link
Member

The monthly events looked a bit odd in this list, so I hid them and added a link to the monthly events on the homepage

Copy link

cloudflare-workers-and-pages bot commented Jan 5, 2025

Deploying dwhdelft-nl with  Cloudflare Pages  Cloudflare Pages

Latest commit: da90ffd
Status: ✅  Deploy successful!
Preview URL: https://12833cbc.dwhdelft-nl-u1a.pages.dev
Branch Preview URL: https://contact-monthly.dwhdelft-nl-u1a.pages.dev

View logs

Copy link

Deploying outsite-nl with  Cloudflare Pages  Cloudflare Pages

Latest commit: da90ffd
Status: ✅  Deploy successful!
Preview URL: https://594e1513.dwhdelft-nl.pages.dev
Branch Preview URL: https://contact-monthly.dwhdelft-nl.pages.dev

View logs

Copy link
Member

@thomcsmits thomcsmits left a comment

Choose a reason for hiding this comment

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

Okay to be honest it took me forever to even see what changed, but I see it now.

Visually this looks better!
But I think there is value in including the Roze Kroeg also in this overview. J&O has it's own page, and no 12-18 y/o is going to be using the page to find it. Same for QT probably, and that isn't as open to public either.

But Roze Kroeg I can imagine people would want to see that here.

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