-
Notifications
You must be signed in to change notification settings - Fork 44
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
respond_to fix should be backported into 4.4.x #14
Comments
Just in case you need it, I created a fork with a tag v4.4.1 https://github.com/Kulgar/spree_legacy_frontend/tags which backports the fix + the migration from turbolinks to turbo. Here are the details about which commits I picked and which I skipped:
|
Personally I don't, because I'm still running 4.3.1 until there is a stable and working version of 4.4+ of both spree and the legacy frontend, which unfortunately there isn't. I hope the Spree team will start taking PRs more seriously because I am left demotivated and saddened by the current situation. I am running with patches for spree_frontend, spree_backend, spree_core, spree_auth_devise and spree_globalize and at the moment it seems to be running smoothly now. Waiting for the fixes to be merged then I will try 4.4+. |
This fix 960003e
Should be put in a stable version. 4.4.0 ("stable" release) is broken due to this.
The text was updated successfully, but these errors were encountered: