Remove SaaS recipe from inner tenants

[hishamco]

Addresses https://github.com/OrchardCMS/OrchardCore/discussions/18396

[Piedone]

Tying the Setup and Tenants modules to a recipe in a theme seems like a very hackish dependency. This much rather needs a properly architected general solution. See https://github.com/OrchardCMS/OrchardCore/issues/16229 for ideas. However, I don't think this is too high of a priority now.

[hishamco]

I have a better solution, but I'd like to keep it simple and allow others to implement SaaS in inner tenants if they need

[Piedone]

Then I'd suggest you present that because I don't think this is appropriate to merge.

[hishamco]

BTW there's a flaky tests in Linux I just notice in my last two PRs

[gvkries]

I agree with @Piedone, it seems to be a hack. I think we should leave it as it is for now, or strive for a complete solution. For example, we could have something in the recipes that connects it with the default tenant. Quite similar to the mentioned issue.

[hishamco]

I agree with @Piedone, it seems to be a hack

I agree, and I elaborate why. Let us hear @sebastienros feedback, or someone mention it in the triage

[github-actions[bot]]

It seems that this pull request didn't really move for quite a while. Is this something you'd like to revisit any time soon or should we close? Please comment if you'd like to pick it up.