-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Stripe Connect and Multi-Shop Checkout #2682
Merged
brent-hoover
merged 62 commits into
marketplace
from
spencer-stripe-connect-merchant-signup
Aug 22, 2017
Merged
Changes from all commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
b355e5e
Add new show/hideForShopTypes setting to the registry
spencern c76caa5
Add provides marketplaceMerchantSettings reactionApp
spencern c6c491b
update provides shopSettings loop to use template instead of name for…
spencern b394dec
add shopAppSwitch helper for showing plugin/app enable switch
spencern 6a58318
Init merchant stripe connect settings
spencern 3fdc173
Unify payments-stripe-connect and payments-stripe plugins
spencern c44a2a9
getPackageSettingsWithOptions method
spencern 7440f93
WIP Stripe Connect Integration
spencern debcc25
Create new shops with account address if it exists
spencern 274cdff
Guard shopType in reactionApps
spencern f9086bc
WIP - Authorize stripe connect. Save connect credentials on merchant'…
spencern 5554816
Direct users to stripe dashboard to check earnings if they have alrea…
spencern a4600f2
Update Reaction.marketplace helper
spencern 0d7b3c3
Update tax rate calculator to be multi-shop aware
spencern f7acd11
Update cart and cartItem schemas to support taxes on each item and ta…
spencern bce643e
add reaction-taxes to the default package list for merchant shops
spencern 22380e6
Add marketplace needed cart transforms.
spencern d1a9487
Fixes for updated taxes methods.
spencern dec3b54
check if merchant's have a stripe account better
spencern 87c9843
WIP Experimental stripe connect multi-payment method
spencern 492d843
Check to make sure that we haven't authorized this merchant yet.
spencern b67bbb8
Successfully charge on behalf of a connected account
spencern ccf4b35
Merge commit 'b99eebc20b78dc335c883b185081aa1aa0457730' into spencer-…
spencern 5b8fdda
Edits to make stripe marketplace method work from checkout
spencern 614e160
Update stripe/createCharges to create a "Payment" object for each cha…
spencern d33915f
[WIP] use call createCharges from client
spencern 22a27c9
Update getApiKey in stripeapi to use primaryShop's stripe acct
spencern d0263ca
update payments schema to store shopId
spencern 351af20
[HACK] Don't check for refunds when sending order notifications
spencern 7716f44
[WIP] Start working on figuring out stripe connect refunds
spencern 6d3f73b
Cleanup
spencern ae2812c
Update cart/submitPayment to work with multiple payments
spencern 459ddb1
Update stripe/createCharges to respect request to "authorize" instead…
spencern 9a69e81
Place order with stripe/createCharges and handle order creation on se…
spencern 5805ac2
Client side validation and confirm that all stripe client safe errors…
spencern 81d4eab
Send `card_error`s back to the client, throw on all other errors. Per…
spencern db95b27
Updates to cart and stripe payment methods to support multi-payments …
spencern b60c6a0
Properly charge primary shop and permit order splitting between prima…
spencern 352aecd
remove comments
spencern 6f54bce
Make standard taxes work with stripe connect / marketplace checkout.
spencern 3f1f300
enable all tests
spencern 8074356
Demo application fee for stripe connect accounts
spencern b382ead
Fix issues in cart transforms
spencern 3fb8697
Adjust packages publications so that we have primary shop routes avai…
spencern f377f59
Don't refresh active shop package publication if active shop is prima…
spencern 4455928
Remove comments
spencern 4715723
Merge branch 'marketplace' into spencer-stripe-connect-merchant-signup
spencern ba68a39
Merge branch 'marketplace' into spencer-stripe-connect-merchant-signup
spencern fdf72e1
Fix issue where order would fail if no taxes are enabled for any shop
spencern e3714db
Update error handling to follow Reaction convention
spencern 9fe8877
Fix issue where shop not having tax data would cause checkout to fail
spencern 7cd3118
Export stripe methods
spencern 5eea68d
Initial tests for updated Stripe package that supports connect
spencern a948a18
Remove old lib stripe api that's no longer used
spencern a96f590
Add nock to package.json
spencern 3f02534
Stub stripe package fetching for marketplace shops in stripe createCh…
spencern f6ca006
Fix stripe create charges test
spencern ee3a42c
Revert unfinished experimental updates to stripe/refunds/list
spencern 9d697c2
export stripe connect methods
spencern c36d24e
Merge branch 'marketplace' into spencer-stripe-connect-merchant-signup
spencern 696cb1e
cast invoice values to numbers before updating cart billing
spencern 0e0f650
Merge branch 'marketplace' into spencer-stripe-connect-merchant-signup
brent-hoover File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we set this variable to options and then just return it, do we actually need it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did that just to make it clear what the options object was, but we don't really need it.