solidusio / solidus_stripe

💳 Integrate Solidus with Stripe
https://stripe.com
BSD 3-Clause "New" or "Revised" License
36 stars 62 forks source link

Consolidate naming for Solidus/Stripe payment methods #283

Closed waiting-for-dev closed 1 year ago

waiting-for-dev commented 1 year ago

Desired Behavior

We have a naming collision between Solidus payment methods & Stripe payment methods. We're not consistent across the code based on how we reference one or the other. That could lead to confusion in some cases.

We can scan the code base and establish a pattern with a relatively low effort.

elia commented 1 year ago

@waiting-for-dev for the sake of clarity can you provide some examples?

waiting-for-dev commented 1 year ago

@waiting-for-dev for the sake of clarity can you provide some examples?

Not sure if the impact is high, but I've been confused in the past, probably mainly in tests.

elia commented 1 year ago

@waiting-for-dev thank you, makes sense, I would say, that living within solidus (aka spree) we should only specify when it's something from stripe, would that work?

waiting-for-dev commented 1 year ago

:100: