Skip to content

Commit ee68aa2

Browse files
chore: remove deprecated FeatureFlagEvaluationTag and TeamDefaultEvaluationTag models (#52186)
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
1 parent 5b7178f commit ee68aa2

File tree

50 files changed

+578
-1049
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

50 files changed

+578
-1049
lines changed

.github/scripts/check-idor-model-coverage.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,6 @@ def get_scoped_models() -> tuple[dict[str, set[str]], set[str]]:
144144
"TeamConversationsSlackConfig",
145145
"TeamCustomerAnalyticsConfig",
146146
"TeamDefaultEvaluationContext",
147-
"TeamDefaultEvaluationTag",
148147
"TeamDataWarehouseConfig",
149148
"TeamMarketingAnalyticsConfig",
150149
"TeamRevenueAnalyticsConfig",

ee/clickhouse/views/experiments.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -444,7 +444,7 @@ def eligible_feature_flags(self, request: Request, **kwargs: Any) -> Response:
444444
- created_by_id: Filter by creator user ID
445445
- order: Sort order field
446446
- evaluation_runtime: Filter by evaluation runtime
447-
- has_evaluation_tags: Filter by presence of evaluation tags ("true" or "false")
447+
- has_evaluation_contexts: Filter by presence of evaluation contexts ("true" or "false")
448448
"""
449449
# validate limit and offset
450450
try:
@@ -469,7 +469,7 @@ def eligible_feature_flags(self, request: Request, **kwargs: Any) -> Response:
469469
created_by_id=request.query_params.get("created_by_id"),
470470
order=request.query_params.get("order"),
471471
evaluation_runtime=request.query_params.get("evaluation_runtime"),
472-
has_evaluation_tags=request.query_params.get("has_evaluation_tags"),
472+
has_evaluation_contexts=request.query_params.get("has_evaluation_contexts"),
473473
)
474474

475475
# Serialize using the standard FeatureFlagSerializer

frontend/src/mocks/handlers.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,6 @@ export const defaultMocks: Mocks = {
216216
'/api/organizations/:organization_id/proxy_records/': [],
217217
'/api/projects/:team_id/dashboard_templates/json_schema/': EMPTY_PAGINATED_RESPONSE,
218218
'/api/organizations/:organization_id/domains/': EMPTY_PAGINATED_RESPONSE,
219-
'/api/environments/:team_id/default_evaluation_tags/': EMPTY_PAGINATED_RESPONSE,
220219
'/api/environments/:team_id/default_evaluation_contexts/': {
221220
default_evaluation_contexts: [],
222221
available_contexts: [],

frontend/src/scenes/experiments/ExperimentForm/SelectExistingFeatureFlagModal.test.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ describe('SelectExistingFeatureFlagModal', () => {
5555
version: 0,
5656
last_modified_by: null,
5757
evaluation_runtime: FeatureFlagEvaluationRuntime.ALL,
58-
evaluation_tags: [],
5958
evaluation_contexts: [],
6059
bucketing_identifier: FeatureFlagBucketingIdentifier.DISTINCT_ID,
6160
}

frontend/src/scenes/experiments/ExperimentForm/VariantsPanelLinkFeatureFlag.test.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ describe('VariantsPanelLinkFeatureFlag', () => {
5656
version: 0,
5757
last_modified_by: null,
5858
evaluation_runtime: FeatureFlagEvaluationRuntime.ALL,
59-
evaluation_tags: [],
6059
evaluation_contexts: [],
6160
bucketing_identifier: FeatureFlagBucketingIdentifier.DISTINCT_ID,
6261
}

frontend/src/scenes/experiments/ExperimentForm/selectExistingFeatureFlagModalLogic.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ describe('selectExistingFeatureFlagModalLogic', () => {
5050
version: 0,
5151
last_modified_by: null,
5252
evaluation_runtime: FeatureFlagEvaluationRuntime.ALL,
53-
evaluation_tags: [],
5453
evaluation_contexts: [],
5554
bucketing_identifier: FeatureFlagBucketingIdentifier.DISTINCT_ID,
5655
},
@@ -88,7 +87,6 @@ describe('selectExistingFeatureFlagModalLogic', () => {
8887
version: 0,
8988
last_modified_by: null,
9089
evaluation_runtime: FeatureFlagEvaluationRuntime.ALL,
91-
evaluation_tags: [],
9290
evaluation_contexts: [],
9391
bucketing_identifier: FeatureFlagBucketingIdentifier.DISTINCT_ID,
9492
},

frontend/src/scenes/experiments/ExperimentForm/selectExistingFeatureFlagModalLogic.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,9 +128,9 @@ export const selectExistingFeatureFlagModalLogic = kea<selectExistingFeatureFlag
128128
offset: filters.page ? (filters.page - 1) * FLAGS_PER_PAGE : 0,
129129
}
130130

131-
// Add evaluation tags filter if required by team
131+
// Add evaluation contexts filter if required by team
132132
if (currentTeam?.require_evaluation_contexts) {
133-
params.has_evaluation_tags = true
133+
params.has_evaluation_contexts = true
134134
}
135135

136136
return params

frontend/src/scenes/experiments/experimentsLogic.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -387,9 +387,9 @@ export const experimentsLogic = kea<experimentsLogicType>([
387387
offset: filters.page ? (filters.page - 1) * FLAGS_PER_PAGE : 0,
388388
}
389389

390-
// Add evaluation tags filter if required by team
390+
// Add evaluation contexts filter if required by team
391391
if (currentTeam?.require_evaluation_contexts) {
392-
params.has_evaluation_tags = true
392+
params.has_evaluation_contexts = true
393393
}
394394

395395
return params

frontend/src/scenes/experiments/utils.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -695,7 +695,6 @@ describe('checkFeatureFlagEligibility', () => {
695695
version: 0,
696696
last_modified_by: null,
697697
evaluation_runtime: FeatureFlagEvaluationRuntime.ALL,
698-
evaluation_tags: [],
699698
evaluation_contexts: [],
700699
bucketing_identifier: FeatureFlagBucketingIdentifier.DISTINCT_ID,
701700
}

frontend/src/scenes/feature-flags/FeatureFlagCodeInstructions.stories.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ const REGULAR_FEATURE_FLAG: FeatureFlagType = {
4242
version: 0,
4343
last_modified_by: null,
4444
evaluation_runtime: FeatureFlagEvaluationRuntime.ALL,
45-
evaluation_tags: [],
4645
evaluation_contexts: [],
4746
bucketing_identifier: FeatureFlagBucketingIdentifier.DISTINCT_ID,
4847
}

0 commit comments

Comments
 (0)