Skip to content

Conversation

@joecohens
Copy link
Contributor

No description provided.

@GrahamCampbell GrahamCampbell changed the title [5.0] Added auth helper [5.0] Added Auth Helper Nov 24, 2014
@JeffreyWay
Copy link
Contributor

I think the name is misleading.

@joecohens
Copy link
Contributor Author

Would you say user() it's better?

@ibrasho
Copy link
Contributor

ibrasho commented Nov 25, 2014

I think user is a better name.

@martinbean
Copy link
Contributor

+1 on using user() instead of auth().

The other helpers tend to return an object of that type, i.e. redirect(). Following this logic, I’d expect auth() to return the authenticator implementation, not the currently authenticated user (if any).

@GrahamCampbell
Copy link
Collaborator

Please squash to one commit.

@joecohens
Copy link
Contributor Author

Sure!

@joecohens joecohens force-pushed the feature/added-auth-helper branch from c0122a8 to f9e5db3 Compare November 25, 2014 20:15
@taylorotwell
Copy link
Member

Hmm, will leave this up to you defining in your own app.

@joecohens
Copy link
Contributor Author

Ok ;)

@joecohens joecohens deleted the feature/added-auth-helper branch January 13, 2015 20:24
@crynobone crynobone mentioned this pull request Mar 16, 2015
@phanan phanan mentioned this pull request Apr 2, 2015
@ghost ghost mentioned this pull request May 20, 2015
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.

6 participants