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
Remove events page #1314
Remove events page #1314
Conversation
Thought we would keep the page (for conferences and Nodeschool events) and just remove the Meetups? |
I agree with @fhemberger with removing the meetups and keeping the page. |
I'm fine with that but then we should update them regularly. I agree with what @Trott wrote here: #1031 (comment). |
Fixes: #1031
5096ab0
to
5891540
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Saw this issue, and logged it in nodejs/community-events#2. That issue there shouldn't be a blocker to this PR, but did want to raise the potential there for people to start getting involved at a future date, should they be interested 👍 |
Not sure how exactly the process works here these days. Can this land? Or is this blocked by objections? |
@Trott We are thinking about keeping some kind of events list, but without the map and the list from meetup.com. Sorry, didn't have time to look into it yet. |
@Trott Okay, we can come up later with a better solution. |
This removes the event pages as per #1031.
The diff is big but it should be easy to review :)