-
Notifications
You must be signed in to change notification settings - Fork 29
Update to Alchemy 8.0 #129
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
Conversation
078cefb to
d905f3f
Compare
We fetch products from it in the alchemy-product-select
The version for rails 7.2 and 8.0
This is the version for Rails 7.2 and 8.0
Convinience for Ruby LSP users
This is cluttering the test logs.
tvdeyen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are changes included that are not related to the Alchemy 8 update, but to alchemy-devise usage. Can we extract those in a separate PR?
app/patches/models/alchemy/solidus/alchemy_devise_ability_patch.rb
Outdated
Show resolved
Hide resolved
|
Separate PR is here: #131 |
We need `Spree.mountedAt` in the product select, and with the move to importmaps we've lost the ability to add stuff successfully to `vendor/assets/alchemy/admin/all.js`. This does the same thing, just a little bit hacky.
tvdeyen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work. And thanks for fixing it
Updates to Alchemy 8.0. Drops 7.4 support.