-
Notifications
You must be signed in to change notification settings - Fork 30
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
Prepare Solidus Sale Prices for Solidus 3.0 #51
Comments
Taking care of this! |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This extension relies on legacy pricing logic and I don't think it can work with Solidus 3.0. In order to move off It can't even modify the price selector to add the API it needs, as that's a configurable class. I'm not actually sure what to do here. As far as I can tell, our stores that use this extension will have to do some hackery to make this work on Solidus 3.0, or just 🤷🏻♂️ and wait for 3.1. |
I guess it only worked with the default price selector before, actually... so maybe it's possible to maintain that since we know what the default selector does. Lame caveat, but I guess it is solvable at least in that case. |
Sorry, just thinking through this: Because it only ever selected prices by currency, we can maintain that for v3.0 and then ideally create a new major release that leverages the pricing selector's new API in v3.1, which would be a big step forward for this gem. |
Update this extension so that it works with Solidus 3.0 by making sure the specs pass.
The text was updated successfully, but these errors were encountered: