Closed (outdated)
Project:
Commerce Stripe
Version:
7.x-3.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
24 Dec 2014 at 23:48 UTC
Updated:
18 Aug 2025 at 18:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
kopeboyComment #2
kopeboyComment #3
kopeboyComment #4
aviindub commentedthats a cool feature, and it probably *could* be rolled in to this module, but i'm not sure it should be. This is distinct enough from the core functionality we are trying to support that it might warrant a whole new module of its own, or at least a submodule underneath this one.
the concept you are describing is also something that is not very well supported in other commerce modules, especially the core module. The company i work for actually has a system like this that we wrote ourselves on top of commerce, but its so convoluted and specialized that its not something that would be useful outside of our application. so there would be some major architecture questions that would have to be addressed beyond just the payment gateway integration.
and then the biggest hurdle -- someone to actually do the work. i know there is zero chance i will personally have the time to write this feature, so unless someone else wants to volunteer to write it, it aint gonna happen. i would not object to including this functionality as a submodule if someone extremely competent wants to step up and write/test it.
I'll leave this open as "postponed" for a while so others can see it.
Comment #5
Sequencing.com commentedI'd also like to +1 this topic - creating a submodule that enables payments to users will be an excellent addition to Commerce Stripe's functionality. Both Stripe and Dwolla's functionality at the non-drupal level already supports this so adding a submodule that will enable this functionality (at least for Stripe) will be great.
Comment #6
torgospizzaThis sounds like Stripe Connect? There was an issue created for that here: #1927694: Stripe Connect but it looks like it kind of dropped off. Seems like that might be the most obvious route.
Comment #7
aviindub commentedi closed the old ticket as duplicate. lets keep the discussion here. you are correct that nothing has happened with it so far.
Comment #8
artusamakI just want to bring your attention on this, i managed to have the Standalone accounts working with fairly minor changes. It actually just cares about having a potential destination attribute sent to Stripe (and using another key). I just didn't handle the Connect accounts link feature as part of the scope of the Connect change (i did it manually).
Would you be ready to move in this direction to merge something like the attached patch or will we have to duplicate most of the code to create a submodule?
Thanks for your feedback.
Comment #9
artusamakFix for the users paying with a credit card that they don't want to store.
Comment #10
goz commentedRe-roll patch and remove destination part.
Destination should be implemented by hook_commerce_stripe_order_charge_alter() which is added by https://www.drupal.org/node/2545400 in 7.x-1.1
Comment #11
torgospizzaI like this idea and am a fan of this functionality. This could be a good first step in getting it implemented.
I'll move to 3.x and we can start to play with it there.
Comment #12
torgospizzaI'll be building the actual "add your bank account" functionality to core Stripe modules for Connect Standalone accounts at #2821521: Connect integration.
I will see if the maintainer of Stripe Managed Accounts would like to potentially integrate with the Stripe module, because I think combining our efforts will result in a better set of modules.
Comment #13
sebastien m. commentedPatch adapted to match 7.x-3.x-dev branch
Comment #14
goz commentedSomething was wrong during the patch adaptation for 3.x branch. platform_key is no more used and condition set the same api.
In any case, this does not apply anymore on 3.x branch.
Reroll the patch
Comment #15
Donit commentedDoes this work?
Comment #16
Donit commentedHas anyone tried this in production?
Comment #17
torgospizzaI haven't gotten a chance to test it yet. Sorry - been focused on other things. I'll try to get to it soon, but it probably won't be until next week sometime.
Comment #18
Donit commentedAlright. Thanks, torgosPizza. I'd be really nice to have a smooth implementation of Connect in the module.
Comment #19
rszrama commentedfwiw, I think this issue is outdated and the patch is non-functional, at least as it compares to my expectation for how Stripe Connect should work. Fortunately, I have someone looking for Stripe Connect functionality that I've broken down into three primary areas:
I don't see anything in the most recent patch that addresses any of the above besides adding a bit of extra config to the payment method form but never using it. I'm tempted to close this out so we can start the conversation over, but I think the best approach will be to leave this open for now and create separate issues against our client requirements instead. We can then close this out if resolving those issues deprecates it.
I'll use a "Connect" tag to group all the issues together for now.
Comment #20
torgospizza@Ryan: I agree. I am open to converting this into a [meta] task and attaching child issues, or just closing this one out and referencing it from a new one if that's preferable.
We have a need for this too, I just haven't been able to set aside the time to handle it properly. But your approach sounds like what we are after too, so I'm looking forward to tackling this, and can test as needed.
Comment #21
Donit commentedThat's great news, Ryan! The requirements sound like full Connect functionality. I'd be more than happy to test it.
Do you have any timeline, client-wise or otherwise, for that task?
Agree: let's move to new issues.
Comment #22
rszrama commentedOur goal is by the end of DrupalCon Nashville being able to support a global site connection to a platform. This won't get us all the way toward full marketplace functionality where individual users could connect their accounts and the site would select the credentials per user, but it would be a start.
We'd have to have a conversation about which other module(s) we'd want to depend on for individual users connecting, but Commerce Marketplace combined with the new hook in Commerce 1.14 for altering payment method info could do the trick. That's out of scope for our particular engagement, but we'll try to keep the broader goal in mind.
Comment #23
Donit commentedI've tested both marketplace modules, Commerce-Marketplace and Commerce Marketplace; the latter offers a better, commerce-compatible logic, but I wasn't able to implement the module successfully in an existing site without getting fatal errors during the checkout. It works in its own ecosystem (installation path), though. My point being: it may be better to bypass a sandbox module.
Comment #24
torgospizzaWe have used Organic Groups in the past, but that might be unnecessary; instead, we might simply go with using Commerce License Role to sell access (or at least, track access through things like Stripe Subscriptions, which avoids having to use Commerce Recurring and the like). So with that in mind I'd be more than happy to test and provide feedback based on our own use case.
Comment #25
farhadhf commentedHey guys, I've resumed working on commerce_marketplace for D7 again and I was looking into writing a module for withdrawals through Stripe Connect and just saw this. Has there been any updates? I can take up the development of the D7 version if you guys haven't worked on it yet, and if you have, I can help make sure it works with commerce_marketplace too.
Comment #26
Donit commentedHi farhadhf, I have a few bits and pieces on this, but I need to resume the work as well and find what I have left. I'd pass it to you. For now: take a look into the Stripe issues tagged with "connect".
Comment #27
tomtech commentedAutomatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.