You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We have two braintree extensions for solidus, the newer one (solidus_paypal_braintree, SPB from now on) is called after the first integration that was available but later grew out of it and now encompasses multiple payment methods. The former (solidus_braintree, SB from now on) has a more sensible name, but is stuck in the past.
What
We should merge both into different versions of SB, more specifically:
merge the history of SPB into the one of SB (add the remote for SPB then merge with a merge-strategy that discards all the old code)
update the namespace and allow people to choose the namespace for database table names
release a new major of SB
release a new major of SPB that is empty and only depends on the newly released SB, with a post-install message that suggests switching the dependency in the gemfile
archive the SPB repo
Considerations
using majors we clearly try to get the attention of the users upgrading and give them a fair chance at understanding what's going on
going forward we'll have less confusion on what extension is the right one for your application
users using a git dependency in their gemfile will still have the burden of checking commits or the changelog manually
What about the SolidusPayPalBraintree namespace and database tables? With this change we would prevent people from switching from the old to the new namespace, unless there's a big database migration.
Premise
We have two braintree extensions for solidus, the newer one (solidus_paypal_braintree, SPB from now on) is called after the first integration that was available but later grew out of it and now encompasses multiple payment methods. The former (solidus_braintree, SB from now on) has a more sensible name, but is stuck in the past.
What
We should merge both into different versions of SB, more specifically:
Considerations
/cc @chrean @kennyadsl
The text was updated successfully, but these errors were encountered: