Skip to content
This repository was archived by the owner on Dec 13, 2018. It is now read-only.

514: Document need for req.user #547

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

514: Document need for req.user #547

mdeggies wants to merge 1 commit into from

Conversation

@mdeggies
Copy link
Member

@mdeggies mdeggies commented Nov 20, 2016 •
edited
Loading

Added a note under the code examples that use req.user. This section links people to 'Resolving the current user' which explains how to use the getUser middleware to access the current user account.

@mdeggies mdeggies mentioned this pull request Nov 20, 2016
Open
@mdeggies mdeggies changed the title 514: Document need for req.user 547: Document need for req.user Nov 20, 2016
@coveralls
Copy link

coveralls commented Nov 20, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 68.536% when pulling 7c717b4 on 514-req-user-middleware into c64c585 on master.

@mdeggies mdeggies changed the title 547: Document need for req.user 514: Document need for req.user Nov 20, 2016
@mdeggies mdeggies changed the title 514: Document need for req.user 514: Document need for req.user Dec 14, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

needs review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mdeggies @coveralls