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

Update CONTRIBUTING with quickstart instructions #855

Closed
wants to merge 1 commit into from

Conversation

nolanlawson
Copy link
Member

These quickstart instructions would have helped me out a lot when I was getting started with Fauxton. Think of it as a "TL;DR, I just want to start contributing now." 😃

These quickstart instructions would have helped me out a lot when I was getting started with Fauxton. Think of it as a "TL;DR, I just want to start contributing now." 😃
@nolanlawson
Copy link
Member Author

One remaining issue, though, is apparently you can't disable Content-Security-Policy in dev without changing devserver.js. Seems like this ought to be configurable by users? Otherwise it's a bit painful, because you can't debug a separate CouchDB/PouchServer running at a different origin.

npm install
```

Next, copy `settings.json.default.json` to `settings.json`. This will be our local settings file.
Copy link
Member

Choose a reason for hiding this comment

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

It will automatically read settings.json.default.json so you only need a settings.json if you have custom code.

/* ... */
"app": {
/* ... */
"host": "http://localhost:5984"
Copy link
Member

Choose a reason for hiding this comment

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

Weird. I thought it would default to this port. Is that not the case?

@nolanlawson
Copy link
Member Author

nolanlawson commented Feb 20, 2017 via email

@garrensmith
Copy link
Member

+1 thanks. I've merged it.

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

2 participants