grahamcampbell / laravel-manager Goto Github PK

Hello, I have a use case where I need to download another project into a temporary folder in my project, update some files and then pull the temporal project back.
My problem is that I can't see how to achieve this using the provided documentation.
I have already set the authentication and the call to GitHub::me() works fine, but when I try to call GitHub::me()->organizations() or GitHub::me()->show("myOrganization","TheRepo"), always returns Github\Exception\RuntimeException with message 'Requires authentication.
How can I get authenticated?

Coming soon to a repo near you.

Hey Graham,
I am currently attempting to write some tests revolving around Vimeo upload functionality. I am using vinkla/laravel-vimeo to accomplish this. During my tests I am attempting to mock the Vimeo facade. The VimeoManger that the facade delivers extends your AbstractManager class. The issue ultimately comes down to the fact that the Mock class expects the __call() method's signature to to be the same.

I have a feeling most classes that use facades don't implement that magic method, so it generally isn't a problem.

The mock class expects the signature:
__call($method, array $args)

whereas AbstractManager has:
__call(string $method, array $parameters)

Attempting to override your method within the VimeoManager doesn't work. However, simply updating your __call() method does. I'm curious if I can create a pull request to literally just change out the signature? I don't see how this would affect any functionality of the method.

Thanks!

@GrahamCampbell - I saw in the past you usually respond quickly, and you're committing to various other repos recently. That leads me to believe that you probably unsubscribed to notifications on PRs for your repos on GitHub.

This is just a friendly reminder that about 5 days ago I sent a couple PRs to your Laravel-Manager and Laravel-DigitalOcean repos on here to bump the version to Laravel 5.7 as it's max, while still retaining the ability for people to update it on 5.5 and 5.6.

Hi, just wondering if you could assist with how to create an FTP connection dynamically, e.g FTP details are stored on a Model and I want to create a new connection at runtime rather than stored in the config/flysystem.php

The facade doesn't implement the createConnection method so I am having trouble figuring it out.

Thanks