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

Connect shop with stripe returns success. In admin UI markeplace owner still sees "Start Accepting Payment" #4989

Closed
TilenKreca opened this issue Feb 15, 2019 · 1 comment
Labels
bug For issues that describe a defect or regression in the released software

Comments

@TilenKreca
Copy link

TilenKreca commented Feb 15, 2019

Issue Description

We are not able to connect our marketplace shop with stripe account. After stripe redirects we receive sucess message but in Admin UI it is not marked as success.

Steps to Reproduce

  1. Primary shop is connected with stripe account successfully.
  2. Markeplace mode was enabled on primary shop and new shop owner was invited.
  3. New shop owner registred account and then connects his account with Stripe My shop settings -> "Start Accepting Payment"
  4. Success response is returned, but users still sees label: "Start Accepting Payment"

Possible Solution

After investigation I found out that in Packages collections there are 2 types stored:

  • reaction-marketplace
  • reaction-stripe

Only reaction-stripe has valid connectAuth under packages collection. If we search reaction-marketplace this property is empty.

If we change value in file:
reaction/imports/plugins/included/markeplace/client/templates/settings/merchant-settings.js

const stripe = Reaction.getPackageSettingsWithOptions({
      shopId: Reaction.getShopId(),
      name: "reaction-marketplace"
    });

slika

to

const stripe = Reaction.getPackageSettingsWithOptions({
      shopId: Reaction.getShopId(),
      name: "reaction-stripe"
    });

then it works, and we get correct info: Your Stripe Account is Connected
slika

So possible solution would be that we would replace name form reaction-marketplace to reaction-stripe in this code: (File: reaction/imports/plugins/included/markeplace/client/templates/settings/merchant-settings.js)

import { Template } from "meteor/templating";

import { Reaction } from "/client/api";

Template.stripeConnectMerchantSignup.helpers({
  stripeConnectIsConnected() {
    const stripe = Reaction.getPackageSettingsWithOptions({
      shopId: Reaction.getShopId(),
      name: "reaction-stripe"
    });
    return stripe && stripe.settings && stripe.settings.connectAuth && stripe.settings.connectAuth.access_token;
  }
});

Is this correct ?

DB comparance: reactin-stripe VS reaction-marketplace

reaction-marketplace
reaction-stripe

Versions

Node: 8.15.0
NPM: 6.7.0
Meteor Node: 8.11.4
Meteor NPM: 6.4.1
Reaction CLI: 0.29.0
Docker: 18.06.1-ce

@brent-hoover brent-hoover added bug For issues that describe a defect or regression in the released software impact-major labels Jun 17, 2019
@aldeed
Copy link
Contributor

aldeed commented Dec 16, 2019

Marketplace stripe is no longer built in

@aldeed aldeed closed this as completed Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug For issues that describe a defect or regression in the released software
Projects
None yet
Development

No branches or pull requests

4 participants