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

docs don't say how to post a payload #59

Closed
mgenev opened this issue Aug 31, 2015 · 12 comments
Closed

docs don't say how to post a payload #59

mgenev opened this issue Aug 31, 2015 · 12 comments

Comments

@mgenev
Copy link

mgenev commented Aug 31, 2015

No offense, but the docs are awful. How do I post a payload? It just gives url as an example...

@analog-nico
Copy link
Member

Hi @mgenev, I see your point. Theoretically, though, all a user has to do is to go to the Request README because Request-Promise exposes the same API. In any case since you had a bad experience with the docs certainly others do, too.

Since you have fresh eyes: In which section of the README would you expect a link to the Request README? In particular I would link the request options section. If that wouldn't be good enough what would you expect from the docs?

@soplakanets
Copy link

+1 for info on how to send data in docs. This is simply a one additional line, right?

@analog-nico
Copy link
Member

Hey guys, I updated the post example in the README. However, since @mgenev found the docs being awful I would appreciate your feedback on how to improve them.

@soplakanets
Copy link

Haha, I wouldn't say they are awful :)

Thanks!

@analog-nico
Copy link
Member

;D

@mgenev
Copy link
Author

mgenev commented Sep 1, 2015

Here's a good example https://visionmedia.github.io/superagent/

I knew how to post right away :D

@yankeeinlondon
Copy link

One thing I found missing was how to set header variables. I imagine this is a very common requirement and I even suspect there's a very easy way to do it but it was not revealed in the docs. :)

@analog-nico
Copy link
Member

@ksnyde Thanks for the tip. I will mention that in the next version of the docs. BTW, all docs of Request are valid for Request-Promise, too. So setting headers is the same.

@yankeeinlondon
Copy link

yeah in the end I did assume that was the case but busy people are stupid people so being explicit pays off more times than not :)

@analog-nico
Copy link
Member

+1

@analog-nico
Copy link
Member

I just released 1.0.0 and with that overhauled the README. Thanks for the feedback all you guys! Let me know if can improve something further.

@mgenev
Copy link
Author

mgenev commented Oct 21, 2015

perfect thanks a lot!

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

No branches or pull requests

4 participants