Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jul 19, 2019. It is now read-only.

Comments.json saving to the correct place #69

Closed
wants to merge 2 commits into from

Conversation

robbyemmert
Copy link

Comments.json would save to the root folder, while the tutorial had us serving it from the /public folder (which is best practice anyway). So this pull request just moves comments.json to the /public folder to avoid confusion. Everything works nicely now :).

@facebook-github-bot
Copy link

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at [email protected]. Thanks!

@robbyemmert
Copy link
Author

I think I signed up successfully. If it didn't go through for some reason, just let me know.

@facebook-github-bot
Copy link

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@zpao
Copy link
Member

zpao commented Aug 18, 2015

I actually want to leave it where it is right now and have it treated more like a database (which is what we're doing anyway, and you'd never put your db in the public folder). I'd much prefer #48 gets done and we have an API endpoint that doesn't look like an actual file name. I appreciate you taking the time though!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants