Skip to content

Laravel Passport#140

Merged
roberthunt merged 8 commits intomasterfrom
passport
May 12, 2017
Merged

Laravel Passport#140
roberthunt merged 8 commits intomasterfrom
passport

Conversation

@dpslwk
Copy link
Member

@dpslwk dpslwk commented May 9, 2017

Now we have updated to 5.4 I've pulled in Passport in order to use it with some JS front end stuff
Its all working nicely with laravel-doctrine thanks to the recent PR in passport

Had to recreate the default laravel migrations in laravel-doctrine/migrations form and the have them excluded from future php artisan doctinre:migrations:diff command, this need the migrations config to be re published.

We are all setup for use with JS and axios as per
https://laravel.com/docs/5.4/passport#consuming-your-api-with-javascript

might need to add the following to app.js for ajax

$.ajaxSetup({
   headers: {
     'X-CSRF-Token': window.Laravel.csrfToken,
   }
});

but my next steps will test that

Will likely need to revisit passport frontend stuff (https://laravel.com/docs/5.4/passport#frontend-quickstart) in the future when I look at hooking it up with discourse and lighting-automation but thats well past base system miles stone needs

@dpslwk dpslwk added this to the Base System milestone May 9, 2017
@roberthunt roberthunt merged commit 4d7ed0b into master May 12, 2017
@dpslwk dpslwk deleted the passport branch May 12, 2017 11:09
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