Page MenuHomePhorge

Only create the necessary minimum of skus for the

Authored by mollekopf on Mar 11 2023, 12:49 PM.
Referenced Files
F10577080: D4142.diff
Wed, Nov 29, 7:24 PM
Unknown Object (File)
Sat, Nov 25, 8:42 AM
Unknown Object (File)
Fri, Nov 24, 3:28 AM
Unknown Object (File)
Thu, Nov 16, 9:30 PM
Unknown Object (File)
Wed, Nov 15, 2:37 AM
Unknown Object (File)
Sun, Nov 12, 8:04 PM
Unknown Object (File)
Wed, Nov 8, 6:50 PM
Unknown Object (File)
Wed, Nov 8, 6:49 PM



In we just create the skus that we require, for the
domain-hosting and beta sku, we adjust the code so we enable the features if
the sku doesn't exist at all.

All sku related migrations have to be dropped from the main migrations,
as the skus are environment specific. I also dropped them from
config.demo, since we already set the appropriate values in the seeder,
so the migrations are pointless.

Original commits:

  • Enable beta entitlements if the beta sku doesn't exist
  • Drop sku related migrations
  • ImapAdminSeeder
  • Enable domain-hosting features if there is no domain-hosting sku
  • Minimal set of skus in the admin seeder

Diff Detail

rK kolab
Lint Not Applicable
Tests Not Applicable

Event Timeline

mollekopf created this revision.
mollekopf added a reviewer: Restricted Project.Mar 11 2023, 12:55 PM

I'll look into the lint problems.

Fixed lint issues (the remaining ones only appear in arc lint)

machniak subscribed.

You removed some core migration files. That means that there will be references to non-existing files in the database (migrations table). I don't know whether this can cause any issues in the future. Maybe not.


This should be Sku::withObjectTenantContext($user).


This should be Sku::withObjectTenantContext($user).


I suppose, if we do such cleanup, this one could also be removed.

This revision now requires changes to proceed.Mar 13 2023, 12:39 PM
This revision is now accepted and ready to land.Mar 22 2023, 11:11 AM
This revision was automatically updated to reflect the committed changes.