Andrea Longhi
Andrea Longhi
@mamhoff thanks for the feedback. I think you have a point about the state machine, though I don't see adding this new state as a big concern... still this solution...
A bit more context on why this can be insidious, again from Solidus sandbox: ```ruby u = Spree::User.create email: '[email protected]', password: 'password', password_confirmation: 'password' o = u.orders.create o.persisted? # =>...
@kennyadsl I've just updated the PR and its description, video included. It's now leveraging Turbo frames and Turbo streams refreshes. Spec failures are related to the latest release of the...
I tracked down the money gem issue to this [PR](https://github.com/RubyMoney/money/pull/1026). The change is included in 6.18.0, but not in 6.17.0, both [released today](https://rubygems.org/gems/money/versions/) 🤷 .
@kennyadsl it should be good for review, I'm sorry for the confusion.
All environments now get listed, but when you try to import to an environment incompatible with the source you will get an error. Still there is room for improvements, such...
@nirnaeth thanks for working on this! ❤️ I've checked out the code, now the API call succeeds with a 200 and the order is not emptied, so no 500 error...
@kennyadsl I confirm you the issue is still present in master.
@kennyadsl yes, that was the issue! Finally it's now green 💚
Interestingly, CI is not running on the PR 🤷