Results 163 comments of Marc Busqué

As it contains code referencing solidus_frontend, we need to move it to the [solidus_frontend installer](https://github.com/solidusio/solidus_frontend/blob/master/lib/generators/solidus_frontend/install/install_generator.rb).

Sure, @piyushswain. Sorry for not being clear enough. Since https://github.com/solidusio/solidus/pull/4490, the Solidus installer prompts the user to select [solidus_frontend](https://github.com/solidusio/solidus_frontend) or [solidus_starter_frontend](https://github.com/solidusio/solidus_starter_frontend) as storefront. The `solidus_bolt` extension injects code to solidus_frontend,...

Thanks, @RyanofWoods! What about a warning in the address factory? Something like: ```ruby after(:build) do |address, evaluator| carmen_country = Carmen::Country.code(adress.country.iso) Spree::Deprecation.warn

Hey @RyanofWoods! > I don't think this warner is aligned with the scenario that I mentioned. The problem is that if they used the Solidus Country factory logic, states_required was...

> are we sure that just having subregions is enough to determine if states are required to make an address valid in a given country? @kennyadsl, it seems that's the...

Hey @RyanofWoods, after talking with the core team, @spaghetticode came up with an idea that we think could solve the concern about breaking tests. What about modifying the address factory...

Hey, @RyanofWoods! :wave: Did you find the time to think about the proposed solution? Do you think it's something that makes sense?

Thanks, @BenMorganIO! Do you think it'd make sense to use the already available [`.checkout_zone` preference](https://github.com/solidusio/solidus/blob/b4ff5ce951654ff070878da6a4ca7fba4418f135/core/lib/spree/app_configuration.rb#L121). Having another `.available_countries` setting is redundant.

I already merged the one for 2.10. As I think it's the only one needed, I'm going to close it. Thanks for working on it!

Oh, I just saw a PR for 2.9. Reopening!!