search

dcblogdev / laravel-microsoft-graph

Laravel package for Microsoft Graph API (Microsoft365)
https://dcblog.dev/docs/laravel-microsoft-graph
Other
120 stars 51 forks source link

Issues with Laravel 9 #46

Closed Kristianmarku closed 1 year ago

Kristianmarku commented 1 year ago

I attempted to integrate it in Laravel 9 following the instructions, but clearly the package has multiple errors for L9. If this repository isn't abandoned, I would appreciate an update to support L9.

MahaEast commented 1 year ago

I’ve only ran intro problems with the migration and listeners. But if You change and add it manually, it works. My problem is the laravel session and user validation. It is possible for a user to login with the last logged in user data. So i have to build my own solution to log in with AD. So i can’t recommend this Module for security reasons.

dcblogdev commented 1 year ago

There is an issue with the migration, I've just pushed a release to change the default migration to an anonymous migration.

I want to update the docs too some of the pages are not as clear as they could be.

Kristianmarku commented 1 year ago

@dcblogdev Thanks for the hard work mate. If you could also update the docs that would be awesome and much appreciated.