Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Service Provider added to README #15

Closed
wants to merge 1 commit into from
Closed

Service Provider added to README #15

wants to merge 1 commit into from

Conversation

mvatansever
Copy link

When I try to install the package with following the README this command not working successfully:

php artisan vendor:publish

Because service provider not added to app/config.php file so I change the README file for non-laravel users :)

@mvatansever
Copy link
Author

mvatansever commented Jul 26, 2017

I just now saw the closed PR for the same reason. This package for L5.5. Apologize for the inconvenience.

@jbrooksuk
Copy link
Member

Horizon works with 5.4. Should be okay to merge this.

@mvatansever
Copy link
Author

mvatansever commented Jul 27, 2017

@jbrooksuk but I noticed in the PR not accepting the same contribution. So I reopen the PR and hope to consider again the pull request.

@mvatansever mvatansever reopened this Jul 27, 2017
@mvatansever
Copy link
Author

This package not seen as will working in L5.4 and before versions because of Connector classes more different than the old versions. So closed the PR for the last time :)

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.

None yet

2 participants