README lists two different types of PayPal credentials #97
Comments
seand7565
added a commit
to seand7565/solidus_paypal_commerce_platform
that referenced
this issue
Oct 6, 2020
You wouldn't normally change these, and they only matter in the context of the wizard - they're not used elsewhere. Unless you're building a marketplace, I don't think there's any reason to adjust these. In addition, we already list PAYPAL_CLIENT information which conflicts with this a bit, so I think to clear up the confusion we should just remove this section from the README. Ref: solidusio#97
seand7565
added a commit
to seand7565/solidus_paypal_commerce_platform
that referenced
this issue
Oct 6, 2020
You wouldn't normally change these, and they only matter in the context of the wizard - they're not used elsewhere. Unless you're building a marketplace, I don't think there's any reason to adjust these. In addition, we already list PAYPAL_CLIENT information which conflicts with this a bit, so I think to clear up the confusion we should just remove this section from the README. Ref: solidusio#97
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The README has the following credentials being set:
PAYPAL_PARTNER_IDPAYPAL_PARTNER_CLIENT_IDPAYPAL_CLIENT_IDPAYPAL_CLIENT_SECRETUnder normal circumstances, these are the ones that should be being set:
PAYPAL_CLIENT_IDPAYPAL_CLIENT_SECRETWe probably don't want people overriding the partner_id information unless they're doing something super crazy, like building a marketplace. I think to avoid confusion (and people not using the default partner information), we should remove the references to the partner_id and partner_client_id from the README, or at the very least explain why they are there and that you probably should avoid setting them.
The text was updated successfully, but these errors were encountered: