Skip to content

Conversation

@amberleyromo
Copy link
Contributor

Closes #6584.

Reorg (a lot of good overview language was already in the plugin authoring page), adding more structure, some language tweaks.

@KyleAMathews
Copy link
Contributor

Deploy preview for using-postcss-sass failed.

Built with commit c853ca9

https://app.netlify.com/sites/using-postcss-sass/deploys/5b6dd09a73f2cf7bcdbabb9b

@gatsbybot
Copy link
Collaborator

Deploy preview for using-drupal ready!

Built with commit c853ca9

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

@gatsbybot
Copy link
Collaborator

Deploy preview for gatsbygram ready!

Built with commit c853ca9

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

@KyleAMathews
Copy link
Contributor

Deploy preview for gatsbyjs failed.

Built with commit c853ca9

https://app.netlify.com/sites/gatsbyjs/deploys/5b6dd09973f2cf7bcdbabb90

Copy link
Contributor

@shannonbux shannonbux left a comment

Choose a reason for hiding this comment

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

This is looking great, @amberleyromo! Just left a few comments, and looping Mike and Michal in to make sure we're updating the plugin authoring doc thoroughly :)


## Build and publish a plugin

For a walkthrough of how to build and publish your own plugin, see the [source plugin tutorial](/docs/source-plugin-tutorial/).
Copy link
Contributor

Choose a reason for hiding this comment

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

would it make sense to link to a stub article(s) for other plugin tutorials (transformer plugin tutorial, for example)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes! Great idea. Let me know if I'm missing any:


Gatsby has a large and growing set of plugins. To search/browse official and
community plugins and their documentation, visit the [Plugin Library](/plugins/).
- add external data or content (e.g. your CMS, static files, a REST API) to your Gatsby GraphQL data
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, this does seem to belong here rather than in /plugin-authoring/. These docs have needed to be re-organized for a long time! What a relief!

@@ -1,15 +1,14 @@
---
title: Plugin Authoring
title: Plugin authoring
Copy link
Contributor

Choose a reason for hiding this comment

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

@m-allanson and @pieh do you remember if there's anything else that needs to be edited about this page? (i.e. common obstacles contributors are running into when they submit plugins?). I remember Mike, you were mentioning recently that you created an issue to help plugin authors signify what version of Gatsby their plugin is compatible with. Or was it the other way around, just helping people vet which plugins work for v2? Either way, wondering if there's something that needs to be edited here in this doc)

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh good idea - here the issue: #7143.

I think we could merge this PR and then make another pass on the plugin docs to mention the Gatsby peerDependency?

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 sure where it would fit in - but maybe this could mention something about learning from the source code of existing plugins?

Most plugins are open source and the code is often quite readable - if you're looking to write your own plugin it can be useful to review the source of similar plugins.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thinking further on this - maybe encourage people to write in a style that others can learn from?

That could be going too far off-track for this doc though. Sometimes people just need to get their stuff done and move on to the next job, so being told how to write your code here could be counter-productive.

Copy link
Contributor

@m-allanson m-allanson left a comment

Choose a reason for hiding this comment

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

Great stuff, thanks @amberleyromo 👍 I left a couple of comments but I think this is good to merge, any followups (if needed) can go in a new PR.

@@ -1,15 +1,14 @@
---
title: Plugin Authoring
title: Plugin authoring
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh good idea - here the issue: #7143.

I think we could merge this PR and then make another pass on the plugin docs to mention the Gatsby peerDependency?

@@ -1,15 +1,14 @@
---
title: Plugin Authoring
title: Plugin authoring
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 sure where it would fit in - but maybe this could mention something about learning from the source code of existing plugins?

Most plugins are open source and the code is often quite readable - if you're looking to write your own plugin it can be useful to review the source of similar plugins.

@@ -1,15 +1,14 @@
---
title: Plugin Authoring
title: Plugin authoring
Copy link
Contributor

Choose a reason for hiding this comment

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

Thinking further on this - maybe encourage people to write in a style that others can learn from?

That could be going too far off-track for this doc though. Sometimes people just need to get their stuff done and move on to the next job, so being told how to write your code here could be counter-productive.

@m-allanson m-allanson merged commit 5a94f06 into gatsbyjs:master Aug 13, 2018
m-allanson added a commit to phacks/gatsby that referenced this pull request Aug 14, 2018
* master: (597 commits)
  Add a site(https://mojaave.com) to showcase list (gatsbyjs#7275)
  feat: create a doc for open source pair programming sessions (gatsbyjs#7266)
  [docs] Add video lesson to the StaticQuery docs (gatsbyjs#7249)
  [v2] docs - update page query docs (gatsbyjs#7285)
  [v2] docs "Styling" overview (gatsbyjs#7288)
  Remove delay (gatsbyjs#7273)
  add site (gatsbyjs#7291)
  Adding new site to the showcase. (gatsbyjs#7281)
  chore(release): Publish
  initial webpack externals support (gatsbyjs#7245)
  add missing package dependencies (gatsbyjs#7259)
  add: custom configuration overview
  (gatsbyjs#7231): tutorial part four updates (gatsbyjs#7240)
  [www] Fix showcase search, checkbox styles (gatsbyjs#7014)
  (gatsbyjs#6584): Restructure plugin overview and plugin authoring pages (gatsbyjs#7229)
  Use Hubspot form for email subscription (gatsbyjs#7233)
  Adding bootstrap CV starter (gatsbyjs#7207)
  Stub Articles and new names (gatsbyjs#7200)
  Improve readability of verbose logging code in wordpress source plugin (gatsbyjs#7146)
  Update hash link to scroll to right section of page (gatsbyjs#7161)
  ...
porfirioribeiro pushed a commit to porfirioribeiro/gatsby that referenced this pull request Aug 22, 2018
@shannonbux shannonbux mentioned this pull request Oct 3, 2018
51 tasks
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.

5 participants