README improvements #21

Merged
merged 1 commit into from Feb 7, 2013

Conversation

Projects
None yet
2 participants
Contributor

tdubourg commented Feb 7, 2013

The require() statements were missing in the code samples.

It was also a bit unclear about where to actually put the requests handling logic (the samples were displaying "hello" + username (in short) and we could thus think that it is a kind of one-time callback...). I just added some comments.

tdubourg Fixing the readme (missing require()) + adding useful comments about …
…where to actually put the request handling logic when authenticating
10f83bd

@gevorg gevorg added a commit that referenced this pull request Feb 7, 2013

@gevorg gevorg Merge pull request #21 from tdubourg/master
README improvements
bc9cad6

@gevorg gevorg merged commit bc9cad6 into http-auth:master Feb 7, 2013

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment