Follow-up from "Added new product rate plan ids"
The following discussion from !2397 should be addressed:
-
@ebaque started a discussion: (+2 comments) NNote for reviewer: I'm not sure we need
SELF_MANAGED_GITLAB_PLANS
here, asPREMIUM_SM_1_YEAR_PLAN
andULTIMATE_SM_1_YEAR_PLAN
are already part ofALL_GITLAB_EE_PLANS
.Speaking of which,
ALL_GITLAB_EE_PLANS
seems to be only read by theSelfHostedUsageNotifier
worker (via a scope). But I can't find any occurrence when greppingSelfHostedUsageNotifier
. Is this worker used anywhere or called from elsewhere? If not, should we remove thisALL_GITLAB_EE_PLANS
constant and do some clean-up?cc @jagood