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

fixed readme to give correct location of file to edit for CMS #103

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

krichey15
Copy link

The current README file is not correct when it tells users to go to "Public/admin/config.yml" this file will be deleted upon running gatsby clean or on every new build.

Users should be going to "static/admin/config.yml" to make permanent changes to the CMS. I correct the readme to say this.

@netlify
Copy link

netlify bot commented Jun 2, 2021

👷 Deploy request for yellowcake pending review.
Visit the deploys page to approve it

🔨 Explore the source changes: c77c13e

@fbuireu
Copy link

fbuireu commented Jun 3, 2021

You are completely right. Next time give credits at least...

Original post where the issue was detected (by me): https://stackoverflow.com/a/67799072/5585371

@krichey15
Copy link
Author

krichey15 commented Jun 3, 2021

Oh I totally would have if I knew I should have. This is the first time I have made any kind of pull request on someone elses Repo.

So in the future should I try to find the stackoverflow user who answered my question(you in this case)on github and give them credit in the pull request comment? I am not asking snidely just trying to learn best practice.

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.

3 participants