Skip to content
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

solidus-4.3 #1

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

solidus-4.3 #1

wants to merge 18 commits into from

Conversation

tvdeyen
Copy link
Owner

@tvdeyen tvdeyen commented Dec 29, 2023

No description provided.

While installing it locally it should be the default as it is
the latest version.
We use at least Alchemy 7.0
1. Run the config in after initialize, so we make sure the menu has been setup by the main app correctly
2. We use the new remix icons if the new admin menu is used.
3. We add our menu tab above the settings icon.
We need to load rspec first in order to get automatic screenshots
The link dialog in Alchemy 7.1 has changed.
It now uses Shoelace Tabs made out of Custom Web Components.

This keeps existing overrides. Note: That will cause a note
in Rails log about not matching overrides. Those can safely be
ignored.
Alchemy 7.1 now allows to submit the form like
any other form with Enter key and prevents submits
correctly.

This still works with Alchemy 6.1 and 7.0 because of
the data attribute we kept on the button tag.
Test with Solidus 4.2 as well
We do not need node anymore
Add deface overrides for Alchemy 7.1
We only support Solidus 4.x now and can finally remove this
complicated setup code.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant