Skip to content

Conversation

alnutile
Copy link
Contributor

@alnutile alnutile commented Sep 5, 2014

that php artisan auth:make outputs


Route: Route::controller('auth', 'Auth\AuthController');
Controller created successfully.
Route: Route::controller('password', 'Auth\RemindersController');
Authentication requests created successfully.
Migration created successfully.
Generating optimized class loader
Compiling common classes
Compiling views

that php artisan auth:make outputs

~~~

Route: Route::controller('auth', 'Auth\AuthController');
Controller created successfully.
Route: Route::controller('password', 'Auth\RemindersController');
Authentication requests created successfully.
Migration created successfully.
Generating optimized class loader
Compiling common classes
Compiling views
~~~
@Anahkiasen
Copy link
Contributor

What ?

@alnutile
Copy link
Contributor Author

alnutile commented Sep 5, 2014

Sorry. What I meant is if you use make:auth it defaults to auth/login for a path. But if you look at the Auth filter it defaults to /login as a path.

@Anahkiasen
Copy link
Contributor

Ah ok. Should be auth/login though, you wrote auth\login

@alnutile
Copy link
Contributor Author

alnutile commented Sep 6, 2014

): long day sorry about that.

On Friday, September 5, 2014, Maxime Fabre notifications@github.com wrote:

Ah ok. Should be auth/login though, you wrote auth\login


Reply to this email directly or view it on GitHub
#2990 (comment).

Al Nutile
http://alfrednutile.info/about
413.345.5103

@alnutile
Copy link
Contributor Author

alnutile commented Sep 7, 2014

closing so I can submit it with the correct slash

@alnutile alnutile closed this Sep 7, 2014
@Anahkiasen
Copy link
Contributor

You could have just pushed to this branch and it'll update the PR

@alnutile
Copy link
Contributor Author

alnutile commented Sep 7, 2014

k, next time I know. For some reason I thought I had to make a new pr.

Thanks!

Al Nutile
http://alfrednutile.info/about
413.345.5103

On Sun, Sep 7, 2014 at 9:22 AM, Maxime Fabre notifications@github.com
wrote:

You could have just pushed to this branch and it'll update the PR


Reply to this email directly or view it on GitHub
#2990 (comment).

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.

2 participants