-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
chore(starters): add gatsby-starter-help-center #20407
Conversation
hey, I'm not a real reviewer here- just a fellow author of gatsby starters who likes checking out new starters! 😃 Nice site @dferber90 !
Then I am able to run all of the npm commands with no issues. 👍 You could consider adding a .npmrc and / or .nvmrc to give a default recommended node version.
You don't have any releases though for this repo on github. It would be nice to publish a 1.0 and have official releases so it's easier for users to track when you make updates the starter and what they entail. One other comment I would have it that I'm not totally sure I know what you mean by "Help Center" so it would be nice to explain more about that in the README somewhere. |
Thanks for your feedback @JimLynchCodes
This happens because your ad blocker is blocking the request to the analytics plugin that comes with the starter. It's nothing to worry about and expected to happen.
People who don't know what a help center is probably won't need this template in the first place. I can add an explanation in case people stumble upon it anyways, thanks for the hint 👍
There are no big changes or new features planned at the moment. When something major comes up, that sounds like a good way to do things! |
@dferber90 Yep, you are right that is was the ad blocker! lgtm! 👍 |
5cfe487
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.
Thanks for the starter! You should absolutely convert this into a Gatsby theme:
https://www.gatsbyjs.org/docs/themes/what-are-gatsby-themes/
Holy buckets, @dferber90 — we just merged your PR to Gatsby! 💪💜 Gatsby is built by awesome people like you. Let us say “thanks” in two ways:
If there’s anything we can do to help, please don’t hesitate to reach out to us: tweet at @gatsbyjs and we’ll come a-runnin’. Thanks again! |
Description
Adds gatsby-starter-help-center to stater library.
You can find a live version and documentation website at help.dferber.de.