Skip to content

Conversation

@stephanbouman
Copy link
Contributor

When installing the package and trying to get connected, my application tells me that I use the wrong argument type in the Storage::extend() method.

Dcblogdev\MsGraph\MsGraphServiceProvider::Dcblogdev\MsGraph\{closure}(): Argument #1 ($app) must be of type string, Illuminate\Foundation\Application given, called in /Users/stephan/Projects/playground/keurmerk-sharepoint/vendor/laravel/framework/src/Illuminate/Filesystem/FilesystemManager.php on line 162 /vendor/dcblogdev/laravel-microsoft-graph/src/MsGraphServiceProvider.php:69	/vendor/laravel/framework/src/Illuminate/Filesystem/FilesystemManager.php:162 in Dcblogdev\MsGraph\MsGraphServiceProvider::Dcblogdev\MsGraph\{closure}()	/vendor/laravel/framework/src/Illuminate/Filesystem/FilesystemManager.php:142 in Illuminate\Filesystem\FilesystemManager::callCustomCreator()	/vendor/laravel/framework/src/Illuminate/Filesystem/FilesystemManager.php:119 in Illuminate\Filesystem\FilesystemManager::resolve()	/vendor/laravel/framework/src/Illuminate/Filesystem/FilesystemManager.php:82 in Illuminate\Filesystem\FilesystemManager::get()	/vendor/laravel/framework/src/Illuminate/Support/Facades/Facade.php:361 in Illuminate\Filesystem\FilesystemManager::disk() 

As stated in the Laravel docs, the extend method accepts the Application instance as first argument.
I changed the return type to Illuminate\Foundation\Application

@dcblogdev dcblogdev merged commit 9028e65 into dcblogdev:master Jun 17, 2025
2 checks passed
@stephanbouman
Copy link
Contributor Author

@dcblogdev Would you like to tag a new release with this change? We'd like to use this in production soon :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants