[solidus:run-tests-on-github-actions] 1 new commit by Thomas von Deyen:

<https://github.com/solidusio/solidus/commit/ceced5c0fd91e1680cf43ff01775207231474b53|ceced5c0fd91e1680cf43ff01775207231474b53>

Only send coverage for Rails 8 and Ruby 3.4 - Thomas von Deyen

[solidusio/solidus] Pull request closed: #6269 Run tests on GitHub actions by tvdeyen

[solidus:main] 5 new commits by Thomas von Deyen:

<https://github.com/solidusio/solidus/commit/98f5c63d891b4c30499918cfd4c0644ab84937b4|98f5c63d891b4c30499918cfd4c0644ab84937b4>

Run tests on github actions - Thomas von Deyen

<https://github.com/solidusio/solidus/commit/f07344d2d1d76711df65a8ffdfbb1b59dd0bdb8d|f07344d2d1d76711df65a8ffdfbb1b59dd0bdb8d>

GH Actions: Add teaspoon backend tests - Thomas von Deyen

<https://github.com/solidusio/solidus/commit/572c4e6147348faf28f9c1d1e25e0bfed68c3c36|572c4e6147348faf28f9c1d1e25e0bfed68c3c36>

Remove bin/build-ci - Thomas von Deyen

<https://github.com/solidusio/solidus/commit/ceced5c0fd91e1680cf43ff01775207231474b53|ceced5c0fd91e1680cf43ff01775207231474b53>

Only send coverage for Rails 8 and Ruby 3.4 - Thomas von Deyen

<https://github.com/solidusio/solidus/commit/009ac6f677304cbc6955801e9bbdc390c8de2360|009ac6f677304cbc6955801e9bbdc390c8de2360>

Merge pull request #6269 from solidusio/run-tests-on-github-actions - Thomas von Deyen

[solidus] The branch "run-tests-on-github-actions" was deleted by tvdeyen

[solidusio/solidus] Issue closed: #4877 app/overrides files loading twice by jarednorman

#6306 Can solidus_backend replace the fucking sassc-rails with other gem? [solidusio/solidus] Issue created by kevin-jj It failed to execute assets:precompile when I used tailwindcss 3.3 and throw exception like below:

Browserslist: caniuse-lite is outdated. Please run:
2.334   npx update-browserslist-db@latest
2.334   Why you should do it regularly: https://github.com/browserslist/update-db#readme
2.354 
2.354 Rebuilding...
2.773 
2.773 Done in 440ms.
4.069 bin/rails aborted!
4.069 SassC::SyntaxError: Error: Function rgb is missing argument $green. (SassC::SyntaxError)
4.069         on line 1389 of stdin
4.069 >>   border-color: rgb(222 222 222 / var(--tw-divide-opacity, 1));

[solidusio/solidus] Issue closed: #6306 Can solidus_backend replace the fucking sassc-rails with other gem? by jarednorman

[solidusio/solidus] Issue closed: #3427 Unify `Spree::User` and `Spree::Order` email validation by jarednorman

[solidusio/solidus] Issue closed: #1449 Order Merging Deleting Orders in Backend Unexpectedly by jarednorman

[solidusio/solidus] Pull request closed: #6304 Create state changes in background job by tvdeyen

[solidus:main] 3 new commits by Thomas von Deyen:

<https://github.com/solidusio/solidus/commit/a3794227d8eec259f9577a9dd52f827278055575|a3794227d8eec259f9577a9dd52f827278055575>

Use ActiveJob::TestHelper - Thomas von Deyen

<https://github.com/solidusio/solidus/commit/2a2df71281612798a0c1b6ef2c45956d289dda5c|2a2df71281612798a0c1b6ef2c45956d289dda5c>

Create state changes in background job - Thomas von Deyen

<https://github.com/solidusio/solidus/commit/b67dbb78353a431b28602f11c888fe85e1937008|b67dbb78353a431b28602f11c888fe85e1937008>

Merge pull request #6304 from blish/background-state-changes - Thomas von Deyen

[solidusio/solidus] Pull request closed: #6280 [Admin] Flashes helper and reorganization by tvdeyen

[solidus:main] 9 new commits by Eugene Chaikin and 1 other:

<https://github.com/solidusio/solidus/commit/7c6fcd6058a80c26867a22210843f5fec1c18622|7c6fcd6058a80c26867a22210843f5fec1c18622>

Add FlashHelper to differentiate between toasts and alerts - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/0292a6c8a5c91d915228910a1e387a92252de194|0292a6c8a5c91d915228910a1e387a92252de194>

Update controllers to use flash[:notice] - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/b191dc3782d69fb69b24cb825c20349d4db31208|b191dc3782d69fb69b24cb825c20349d4db31208>

Update application view to render

toasts

from helper - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/c02a105d4a0cdaf565658181ba27e84b607320f3|c02a105d4a0cdaf565658181ba27e84b607320f3>

Add layout container for alerts - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/de123153de8591b3fcc3ee0ebf717921191b563f|de123153de8591b3fcc3ee0ebf717921191b563f>

Move flashes to dedicated layout components - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/0bd1b35194a1bd57715c99a701f70f2849a7ac92|0bd1b35194a1bd57715c99a701f70f2849a7ac92>

Support default alert titles - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/2526156a9c68aca8a65337fb2ffa75d5621e67ec|2526156a9c68aca8a65337fb2ffa75d5621e67ec>

Support flash[:alert] = <String> - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/27adf3de0fd1d922391b6ce69c05dea9a7153ea9|27adf3de0fd1d922391b6ce69c05dea9a7153ea9>

Update alert attribute "description" to "message" - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/008ef4d3468c631b4f0a981dc330ac1c025ca789|008ef4d3468c631b4f0a981dc330ac1c025ca789>

Merge pull request #6280 from chaimann/admin-flash-helper - Thomas von Deyen

[solidusio/solidus] Pull request closed: #6303 Restrict deletion of adjustment reasons if adjustments present by tvdeyen

[solidus:main] 3 new commits by Martin Meyerhoff and 1 other:

<https://github.com/solidusio/solidus/commit/e6196717f6bd268e90f4e7d575f75b3d68362ac0|e6196717f6bd268e90f4e7d575f75b3d68362ac0>

AdjustmentReason: Add :dependent option - Martin Meyerhoff

<https://github.com/solidusio/solidus/commit/9e1ca7f195f4c7fe9c0cee0153d420e41a8f40dc|9e1ca7f195f4c7fe9c0cee0153d420e41a8f40dc>

Add FK between Adjustments and Adjustment Reasons - Martin Meyerhoff

<https://github.com/solidusio/solidus/commit/13a7bbd6e54f978a1bc064d43dfca67901a3d9a3|13a7bbd6e54f978a1bc064d43dfca67901a3d9a3>

Merge pull request #6303 from mamhoff/adjustment-reason-dependent - Thomas von Deyen

[solidusio/solidus] Pull request closed: #6237 fix admin component generator namespace by tvdeyen

[solidus:main] 2 new commits by César Carruitero and 1 other:

<https://github.com/solidusio/solidus/commit/c742a293b21bbd8268f68a6de9b76eaba4e6740b|c742a293b21bbd8268f68a6de9b76eaba4e6740b>

fix admin component generator namespace - César Carruitero

<https://github.com/solidusio/solidus/commit/48de765727938880dda51b3ea17c2503dc6423a3|48de765727938880dda51b3ea17c2503dc6423a3>

Merge pull request #6237 from ccarruitero/admin_generator - Thomas von Deyen

[solidus] New branch "backport/v4.5/pr-6237" was pushed by github-actions[bot]

#6307 [v4.5] fix admin component generator namespace [solidusio/solidus] Pull request submitted by github-actions[bot] Backport This will backport the following commits from

main

to `v4.5`: • Merge pull request #6237 from ccarruitero/admin_generator <!--- Backport version: 9.4.3 --> Questions ? Please refer to the Backport tool documentation GitHub - sorenlouv/backport: A simple CLI tool that automates the process of backporting commits on a GitHub repo A simple CLI tool that automates the process of backporting commits on a GitHub repo - sorenlouv/backport

[solidusio/solidus] Pull request closed: #6160 Admin stock locations form by tvdeyen

[solidus:main] 13 new commits by Eugene Chaikin and 1 other:

<https://github.com/solidusio/solidus/commit/7bb2a5a91c20c79678e681030f089557d2bd4a5e|7bb2a5a91c20c79678e681030f089557d2bd4a5e>

Add missing "Add new" button on the stock locations index page - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/49eeb96f210419c2038c6980692cdc898ab3b877|49eeb96f210419c2038c6980692cdc898ab3b877>

Update stock locations index table to use explicit links - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/fee63a2c6530612329b1c9896382af917bbd58a9|fee63a2c6530612329b1c9896382af917bbd58a9>

Update stock locations controller - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/8a49ab554bfa92511c6f96a27033d86979aa3fb9|8a49ab554bfa92511c6f96a27033d86979aa3fb9>

Rename resource form turbo frame - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/ef931b8996be37f9ff68f193409a7ce02d29805d|ef931b8996be37f9ff68f193409a7ce02d29805d>

Add page with form for new stock location - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/24639d47dbef98b9a6ed098c70d410ccb024800f|24639d47dbef98b9a6ed098c70d410ccb024800f>

Move stock location form into a separate component - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/977e137e4f12f5a8d56eb8fbba6737b103e845dd|977e137e4f12f5a8d56eb8fbba6737b103e845dd>

Allow stock location edits with new form - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/c2ec3222303d27742fec2f45e49bba3a704f1fc0|c2ec3222303d27742fec2f45e49bba3a704f1fc0>

Add helper methods for commonly used UI buttons - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/ed03277a74ffa7c670f67c6fb15cfdfe2b91c1f4|ed03277a74ffa7c670f67c6fb15cfdfe2b91c1f4>

Add specs for stock location features and requests - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/11065427484196094716ffcf52c55691f02bc9d6|11065427484196094716ffcf52c55691f02bc9d6>

Create default fieldset "contact_info" - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/80b4f2c02f7b1307562d61445b06c76bd998536e|80b4f2c02f7b1307562d61445b06c76bd998536e>

Add Contact information section on stock locations form - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/8f9d2b25331ebb538f8db8631fd0b9d69aed60c6|8f9d2b25331ebb538f8db8631fd0b9d69aed60c6>

Remove turbo frame tags from new and edit pages - Eugene Chaikin

<https://github.com/solidusio/solidus/commit/d67ef3e78100330a92c90b3ca4a52bdd2b7d2819|d67ef3e78100330a92c90b3ca4a52bdd2b7d2819>

Merge pull request #6160 from chaimann/admin-stock-locations-form - Thomas von Deyen

#6308 Match foreign key constraint error causes on exception names [solidusio/solidus] Pull request submitted by mamhoff Summary We've added pretty sophisticated error handling to the migrations that add foreign keys. When I first hit one of these errors, though, I found that in a Solidus context, only the error classes for the currently used database adapter will be loaded. Because our migrations need to work with all adapters, this PR changes the matching to be on the string name of the exception, rather than on the exception class. Additionally, this changes the messaging to use

ActiveRecord::Migration#say

rather than directly invoking

Rails.logger

. Checklist Check out our PR guidelines for more details. The following are mandatory for all PRs: • [x] I agree that my PR will be published under the same license as Solidus. • [x] I have written a thorough PR description. • [x] I have kept my commits small and atomic. • [x] I have localized any and all user-facing strings that I added to the source code. • [x] I have used clear, explanatory commit messages. The following are not always needed: • 📖 I have updated the README to account for my changes. • 📑 I have documented new code with YARD. • 🛣️ I have opened a PR to update the guides. • ✅ I have added automated tests to cover my changes. • 📸 I have attached screenshots to demo visual changes. solidus/LICENSE.md at main · solidusio/solidus 🛒 Solidus, the open-source eCommerce framework for industry trailblazers. - solidusio/solidus .github/CONTRIBUTING.md at main · solidusio/.github Contribute to solidusio/.github development by creating an account on GitHub.

[solidusio/solidus] Pull request closed: #6308 Match foreign key constraint error causes on exception names by tvdeyen

[solidus:main] 3 new commits by Martin Meyerhoff and 1 other:

<https://github.com/solidusio/solidus/commit/9f0c1ed464a9c91a094149e9ea26acf9ae10591e|9f0c1ed464a9c91a094149e9ea26acf9ae10591e>

FK Migrations: Match on Exception names - Martin Meyerhoff

<https://github.com/solidusio/solidus/commit/707590cb71949fb9e24673b9b26730a3f703db34|707590cb71949fb9e24673b9b26730a3f703db34>

FK migration exception handling: Use "say" - Martin Meyerhoff

<https://github.com/solidusio/solidus/commit/424f28437d2c25d60b2a48c041f83baa62265222|424f28437d2c25d60b2a48c041f83baa62265222>

Merge pull request #6308 from mamhoff/db-adapter-rescue-strings - Thomas von Deyen

#6309 Add foreign key and handling for stock location deletion to customer returns [solidusio/solidus] Pull request submitted by mamhoff Summary This PR adds a foreign key constraint such that all customer returns have a stock location ID that refers to an existing stock location. It also restricts deleting stock locations with customer returns present, as customer returns frequently also hold information about reimbursements and generally reflect real-life occurrences. In order to provide an o-kay interface deleting a stock location with customer returns present: <img width="760" height="262" alt="grafik" src="https://github.com/user-attachments/assets/86c4c010-b961-4bc5-82e2-84d552d94e3f" /> (This is the legacy admin interface,

solidus_admin

does not have a button for deleting stock locations). Checklist Check out our PR guidelines for more details. The following are mandatory for all PRs: • [x] I agree that my PR will be published under the same license as Solidus. • [x] I have written a thorough PR description. • [x] I have kept my commits small and atomic. • [x] I have localized any and all user-facing strings that I added to the source code. • [x] I have used clear, explanatory commit messages. The following are not always needed: • 📖 I have updated the README to account for my changes. • 📑 I have documented new code with YARD. • 🛣️ I have opened a PR to update the guides. • ✅ I have added automated tests to cover my changes. • 📸 I have attached screenshots to demo visual changes. solidus/LICENSE.md at main · solidusio/solidus 🛒 Solidus, the open-source eCommerce framework for industry trailblazers. - solidusio/solidus .github/CONTRIBUTING.md at main · solidusio/.github Contribute to solidusio/.github development by creating an account on GitHub. GitHub - solidusio/edgeguides: Developer documentation for the Solidus eCommerce framework. Developer documentation for the Solidus eCommerce framework. - solidusio/edgeguides

#6310 Setup generator namespace in generator [solidusio/solidus] Pull request submitted by mamhoff Summary #6237 introduced setting the generator namespace in an initializer - however, that broke the sandbox and

rails server

, because in that initializer

Rails::Generator

would only be defined if we're actually running a generator, but not when running

rails server

or

rails console

. This PR reverts that commit, and instead proposes moving setting the generator namespace to the generator itself. That way, we control a little better what's going on. I'm not sure this is the best solution out there. My questions are: • do we need to namespace things in

SolidusAdmin

? Should we assume that all components ever defined for the new admin live in that namespace? • How is

Rails::Generators.namespace

usually set? Should we even set it? I do not have answers to these questions. But this PR allows us to run both the component generator and

rails server

. Checklist Check out our PR guidelines for more details. The following are mandatory for all PRs: • [x] I agree that my PR will be published under the same license as Solidus. • [x] I have written a thorough PR description. • [x] I have kept my commits small and atomic. • [x] I have localized any and all user-facing strings that I added to the source code. • [x] I have used clear, explanatory commit messages. The following are not always needed: • 📖 I have updated the README to account for my changes. • 📑 I have documented new code with YARD. • 🛣️ I have opened a PR to update the guides. • ✅ I have added automated tests to cover my changes. • 📸 I have attached screenshots to demo visual changes. solidus/LICENSE.md at main · solidusio/solidus 🛒 Solidus, the open-source eCommerce framework for industry trailblazers. - solidusio/solidus .github/CONTRIBUTING.md at main · solidusio/.github Contribute to solidusio/.github development by creating an account on GitHub. GitHub - solidusio/edgeguides: Developer documentation for the Solidus eCommerce framework. Developer documentation for the Solidus eCommerce framework. - solidusio/edgeguides

#6311 Use FactoryBot factories [solidusio/solidus] Pull request submitted by benjaminwil Summary In our work on the in-memory order updater (#5872), we updated some tests to have more realistic test setup. This is not dependent on that work, so we are splitting it into its own small pull request. Instead of instantiating new records directly or using

mock_model

, we can use factories, which are more realistically set up objects. Checklist Check out our PR guidelines for more details. The following are mandatory for all PRs: • [x] I agree that my PR will be published under the same license as Solidus. • [x] I have written a thorough PR description. • [x] I have kept my commits small and atomic. • [x] I have localized any and all user-facing strings that I added to the source code. • [x] I have used clear, explanatory commit messages. solidus/LICENSE.md at main · solidusio/solidus 🛒 Solidus, the open-source eCommerce framework for industry trailblazers. - solidusio/solidus .github/CONTRIBUTING.md at main · solidusio/.github Contribute to solidusio/.github development by creating an account on GitHub.

#6312 Use new configuration setting for item total class [solidusio/solidus] Pull request submitted by benjaminwil Summary In our work on the in-memory order updater (#5872), we created a new configuration point

item_total_class

, but we did not use this configuration class in the

solidus_legacy_promotions

gem. This pull request simply starts using the configuration point instead of calling the built-in

Spree::ItemTotal

class directly. Checklist Check out our PR guidelines for more details. The following are mandatory for all PRs: • [x] I agree that my PR will be published under the same license as Solidus. • [x] I have written a thorough PR description. • [x] I have kept my commits small and atomic. • [x] I have localized any and all user-facing strings that I added to the source code. • [x] I have used clear, explanatory commit messages. solidus/LICENSE.md at main · solidusio/solidus 🛒 Solidus, the open-source eCommerce framework for industry trailblazers. - solidusio/solidus .github/CONTRIBUTING.md at main · solidusio/.github Contribute to solidusio/.github development by creating an account on GitHub.

#6313 Set required line item attributes earlier [solidusio/solidus] Pull request submitted by benjaminwil Summary This change makes working with unpersisted line items easier because required attributes are set immediately. This change is in service of creating an in-memory order updater (#5872) but is not dependent on that work and can be merged as a separate small pull request. Checklist Check out our PR guidelines for more details. The following are mandatory for all PRs: • [x] I agree that my PR will be published under the same license as Solidus. • [x] I have written a thorough PR description. • [x] I have kept my commits small and atomic. • [x] I have localized any and all user-facing strings that I added to the source code. • [x] I have used clear, explanatory commit messages. solidus/LICENSE.md at main · solidusio/solidus 🛒 Solidus, the open-source eCommerce framework for industry trailblazers. - solidusio/solidus .github/CONTRIBUTING.md at main · solidusio/.github Contribute to solidusio/.github development by creating an account on GitHub.

[solidusio/solidus] Pull request closed: #6311 Use FactoryBot factories by benjaminwil