search

l4nos / laravel-cashier-stripe-connect

💲 Adds Stripe Connect functionality to Laravel's main billing package, Cashier. Simply works as a drop-in on top of Cashier, with no extra configuration.
https://updev-1.gitbook.io/cashier-for-connect/
MIT License
62 stars 27 forks source link

Cashier Requirement Installation #12

Closed kornatzky closed 1 year ago

kornatzky commented 1 year ago

Here is what I see in Laravel 10 project. With cashier just installed:

Problem 1

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions. You can also try re-running composer require with an explicit version constraint, e.g. "composer require lanos/laravel-cashier-stripe-connect:*" to figure out if any version is installable, or "composer require lanos/laravel-cashier-stripe-connect:^2.1" if you know which you need.

kornatzky commented 1 year ago

composer.json:

"laravel/framework": "^10.0", "laravel/cashier": "^14.9",

kornatzky commented 1 year ago

Seems a simple change in the requirements from 14.6 to 14.9 would do it

kornatzky commented 1 year ago

on checking it is more complicated

kornatzky commented 1 year ago

We get:

- lanos/laravel-cashier-stripe-connect dev-main requires illuminate/console ^8.37|^9.0|^v10.3.3 -> found illuminate/console[v8.37.0, ..., 8.x-dev, v9.0.0-beta.1, ..., 9.x-dev, v10.3.3, 10.x-dev] but these were not loaded, likely because it conflicts with another require.

NickSugden commented 1 year ago

There's a pull request (#13 ) in that resolves this issue which is requiring @l4nos to merge. I did create a fork of this project which you can use while waiting for the merge, or to allow for anyone wanting to validate these changes fix the issue. composer require nicksugden/laravel-cashier-stripe-connect-l10

kornatzky commented 1 year ago

Great. Thanks @NickSugden . Going to try it.

kornatzky commented 1 year ago

Installation is fine. Thanks for the help again @NickSugden Bravo!

madewithrunway commented 1 year ago

I will merge this within 24 hours. Apologies for the delay!

phillmorgan28 commented 1 year ago

Any update on when this will be merged?

l4nos commented 1 year ago

This has been merged. Sorry for the delay, there is a new release too v.1.0.14 which contains "on_behalf_of" functionality on destination charges too.