Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
924b7d24
Commit
924b7d24
authored
Apr 04, 2022
by
Kerri Miller
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Remove usage_data_diff_searches feature flag
Changelog: removed
parent
3ddbc0d0
Changes
4
Hide whitespace changes
Inline
Side-by-side
Showing
4 changed files
with
2 additions
and
14 deletions
+2
-14
app/assets/javascripts/diffs/components/app.vue
app/assets/javascripts/diffs/components/app.vue
+2
-4
app/controllers/projects/merge_requests_controller.rb
app/controllers/projects/merge_requests_controller.rb
+0
-1
config/feature_flags/development/usage_data_diff_searches.yml
...ig/feature_flags/development/usage_data_diff_searches.yml
+0
-8
lib/gitlab/usage_data_counters/known_events/code_review_events.yml
...b/usage_data_counters/known_events/code_review_events.yml
+0
-1
No files found.
app/assets/javascripts/diffs/components/app.vue
View file @
924b7d24
...
...
@@ -534,10 +534,8 @@ export default {
if
(
delta
>=
0
&&
delta
<
1000
)
{
this
.
disableVirtualScroller
();
if
(
window
.
gon
?.
features
?.
usageDataDiffSearches
)
{
api
.
trackRedisHllUserEvent
(
'
i_code_review_user_searches_diff
'
);
api
.
trackRedisCounterEvent
(
'
diff_searches
'
);
}
api
.
trackRedisHllUserEvent
(
'
i_code_review_user_searches_diff
'
);
api
.
trackRedisCounterEvent
(
'
diff_searches
'
);
}
}
});
...
...
app/controllers/projects/merge_requests_controller.rb
View file @
924b7d24
...
...
@@ -47,7 +47,6 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo
# Usage data feature flags
push_frontend_feature_flag
(
:users_expanding_widgets_usage_data
,
project
,
default_enabled: :yaml
)
push_frontend_feature_flag
(
:diff_settings_usage_data
,
default_enabled: :yaml
)
push_frontend_feature_flag
(
:usage_data_diff_searches
,
project
,
default_enabled: :yaml
)
end
before_action
do
...
...
config/feature_flags/development/usage_data_diff_searches.yml
deleted
100644 → 0
View file @
3ddbc0d0
---
name
:
usage_data_diff_searches
introduced_by_url
:
rollout_issue_url
:
milestone
:
'
14.2'
type
:
development
group
:
group::code review
default_enabled
:
true
lib/gitlab/usage_data_counters/known_events/code_review_events.yml
View file @
924b7d24
...
...
@@ -246,7 +246,6 @@
redis_slot
:
code_review
category
:
code_review
aggregation
:
weekly
feature_flag
:
usage_data_diff_searches
-
name
:
i_code_review_total_suggestions_applied
redis_slot
:
code_review
category
:
code_review
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment