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

Fancy Javascript Example #1492

Merged
merged 4 commits into from Jul 18, 2017
Merged

Conversation

jbolda
Copy link
Contributor

@jbolda jbolda commented Jul 13, 2017

There is still stuff I could add (especially to the README), but wanted to get this out before I had to jump back to paid work. I tried to comment the files themselves well at least.

@KyleAMathews
Copy link
Contributor

KyleAMathews commented Jul 13, 2017

@gatsbybot
Copy link
Collaborator

gatsbybot commented Jul 13, 2017

Deploy preview ready!

Built with commit 0c41941

https://deploy-preview-1492--gatsbyjs.netlify.com

@gatsbybot
Copy link
Collaborator

gatsbybot commented Jul 13, 2017

Deploy preview ready!

Built with commit 0c41941

https://deploy-preview-1492--using-drupal.netlify.com

@gatsbybot
Copy link
Collaborator

gatsbybot commented Jul 13, 2017

Deploy preview ready!

Built with commit 0c41941

https://deploy-preview-1492--gatsbygram.netlify.com

Copy link
Contributor

@KyleAMathews KyleAMathews left a comment

Choose a reason for hiding this comment

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

Looking great!

Could you change the name of the site to something more "boring" but descriptive e.g. using-javascript-frontmatter. That'll make the site more discoverable.

@@ -0,0 +1,4 @@
{
"presets": ['react', 'es2015', 'stage-0'],
Copy link
Contributor

Choose a reason for hiding this comment

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

This and the .eslintrc file shouldn't be necessary

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Gatsbygram had them so I figured I would include them. Can remove them though.

"bulma": "0.4.2",
"d3": "4.9.1",
"gatsby": "^1.1.0",
"gatsby-link": "^1.1.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Many of these plugins don't have a 1.1.0 release yet — Lerna doesn't make a new release for a package unless there's been changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Haha, I was wondering, yarn makes it to easy to ignore this fact.

@jbolda
Copy link
Contributor Author

jbolda commented Jul 13, 2017

So my expectation with this example is to showcase any plugin/transformer/etc. that deals directly with javascript. For now, it is just frontmatter, yes. But as we add more js stuff, I would presume that it will be a little bit more complicated then say, markdown, to compose and it feels like it would be nice have an example documenting how they interact / work together. (I mostly showcased the markdown as an addition to this as it is painful to write directly in javascript, and the convenience of markdown I presume will make it a very typical pair.)

That being said, I am not opposed to a name change, but I think it would be worthwhile to keep it a little bit more... general I guess? What are your thoughts?

@KyleAMathews
Copy link
Contributor

So right now we have three types of example sites — full-blown applications like gatsbygram, very small plugin example sites, and one "ecosystem" example site https://using-remark.gatsbyjs.org/

I can see this site also becoming a kind of "ecosystem" site. If that's your goal, perhaps name the site "using-javascript-transforms"?

@jbolda
Copy link
Contributor Author

jbolda commented Jul 13, 2017

Yea, definitely, that sounds perfect to me!

@jbolda
Copy link
Contributor Author

jbolda commented Jul 14, 2017

I think that covers everything.

@KyleAMathews
Copy link
Contributor

Awesome! Thanks so much Jacob!

@KyleAMathews KyleAMathews merged commit d249bc0 into gatsbyjs:master Jul 18, 2017
@KyleAMathews
Copy link
Contributor

Hey just tried building this on Netlify and got this error:

12:50:29 PM: Error: ReferenceError: location is not defined
at SiteSidebar.render (render-page.js:64188:21)

Could you look into it and PR a fix?

@jbolda
Copy link
Contributor Author

jbolda commented Jul 18, 2017

Will do!

@jbolda jbolda deleted the fancy-javascript branch August 12, 2017 02:36
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.

None yet

3 participants