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

Create attend-events.js #679

Closed
wants to merge 2 commits into from
Closed

Create attend-events.js #679

wants to merge 2 commits into from

Conversation

veextee
Copy link
Contributor

@veextee veextee commented Aug 25, 2020

Attempt at taking Stijn's kubecon page and jemiah's nerd days page to make an "attend events" page. please edit.

@CLAassistant
Copy link

CLAassistant commented Aug 25, 2020

CLA assistant check
All committers have signed the CLA.

@jpvajda jpvajda self-requested a review August 25, 2020 01:02
@jpvajda jpvajda added events work in progress This is work that is not yet ready for review labels Aug 25, 2020
@jpvajda
Copy link
Contributor

jpvajda commented Aug 25, 2020

Relates to #678

@jerelmiller jerelmiller linked an issue Aug 25, 2020 that may be closed by this pull request
Copy link
Contributor

@jerelmiller jerelmiller left a comment

Choose a reason for hiding this comment

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

@veextee I made a couple pretty minor comments. Thanks so much for creating this! This will be super nice to have!!

@@ -0,0 +1,71 @@
import React from 'react';
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not quite sure how to comment on the file as a whole, but just wanted to point out that you have this file in the root folder. Unfortunately Gatsby won't be able to see this file, so it won't actually create a page for it. Can you move this file to the src/pages directory? Gatsby will then be able to pick it up, work some magic ✨ and create a page for it.

From what I can tell otherwise, this page looks pretty good 🙂

Co-authored-by: Jerel Miller <[email protected]>
@jerelmiller
Copy link
Contributor

Closing in favor of #684

@jerelmiller jerelmiller deleted the attend-events branch August 25, 2020 17:05
@jpvajda jpvajda removed the work in progress This is work that is not yet ready for review label Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a landing page for Attend events
4 participants