solidusio / solidus_paypal_commerce_platform

💳 Integrate Solidus with Paypal Commerce Platform
https://developer.paypal.com/docs/platforms/
BSD 3-Clause "New" or "Revised" License
17 stars 24 forks source link

Uniform installer with Solidus Starter Frontend #184

Closed kennyadsl closed 1 year ago

kennyadsl commented 1 year ago

Summary

Solidus installer uses the --frontend=starter/classic format to choose which frontend to install. This is an attempt to try to use the same format to be uniform and provide some predictability to our users.

The installer code is a bit more complex, though.

For some context, this comes from me trying to do something similar in https://github.com/solidusio/solidus_subscriptions/pull/276: as pointed out, we might try to create a standard, uniform way of solving this problem.

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed (~cross them out~ if they are not):

gsmendoza commented 1 year ago

Nitpick. Typo on first commit message:

This will streamline development, being able to reflect changes to the sandbox immediately (regenerating it without other tweaks).