Skip to content

Admin UI fixes #176

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

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

Admin UI fixes #176

wants to merge 8 commits into from

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented May 24, 2025

Needs #175

This extensions Admin UI has not been updated ever. Solidus has store admin templates included since forever.
Since this extension adds a lot of fields to the admin forms I kept them, but updated them to match the
current (4.5) Solidus backend UI.

Before

Screenshot 2025-05-24 at 10-51-05 Editing Store

After

Screenshot 2025-05-24 at 10-57-23 Editing Store - Stores

tvdeyen added 8 commits May 23, 2025 20:22
Not everybody uses the starter frontend.
Rails does not create a sprockets manifest anymore,
but we still depend on it. Since rails commands do not
work anymore, because sprockets raises an error if no
`manifest.js` file is present we need to create it in
order to be able to run `rails new`.
Solidus main already has a stores configuration tab.
This has not been updated in a long while.
Although Solidus has its own Stores index this
extenstion made some important aditions:

1. It allows to destroy stores
2. It adds mail from adress and code

That qualifies this template override
Solidus has a form as well. Since this extension
added a lot of fields, we update the override to
the latest of Solidus 4.5
solidus_dev_support removed Rubocop and uses
Standard for formatting rules.
Rails does not create a Sprockets manifest anymore.
Need to create it before being able to run any
rails commands.
@tvdeyen tvdeyen mentioned this pull request May 26, 2025
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