Still enable find owners even if code owners enabled
Change-Id: I95bc2efed44300e5c1160da82c77b4017423d385
diff --git a/src/main/resources/static/find-owners.js b/src/main/resources/static/find-owners.js
index 26c4a8f..d2c0c97 100644
--- a/src/main/resources/static/find-owners.js
+++ b/src/main/resources/static/find-owners.js
@@ -13,17 +13,20 @@
// limitations under the License.
Gerrit.install(function(self) {
- const ENABLED_EXPERIMENTS = window.ENABLED_EXPERIMENTS || [];
- if (ENABLED_EXPERIMENTS.includes('UiFeature__plugin_code_owners')) {
- // Disable if experiment UiFeature__plugin_code_owners is enabled
- // Also hide previous 'Find Owners' button under 'MORE'.
- function removeFindOwnerFromSecondaryActions() {
- var changeActions = self.changeActions();
- changeActions.setActionHidden('revision', 'find-owners~find-owners', true);
- }
- self.on('showchange', removeFindOwnerFromSecondaryActions);
- return;
- }
+ // TODO: ideally we want to turn off find owners when code owners enabled
+ // but since they are two plugins right now, we just keep find owners around
+ // for a while
+ // const ENABLED_EXPERIMENTS = window.ENABLED_EXPERIMENTS || [];
+ // if (ENABLED_EXPERIMENTS.includes('UiFeature__plugin_code_owners')) {
+ // // Disable if experiment UiFeature__plugin_code_owners is enabled
+ // // Also hide previous 'Find Owners' button under 'MORE'.
+ // function removeFindOwnerFromSecondaryActions() {
+ // var changeActions = self.changeActions();
+ // changeActions.setActionHidden('revision', 'find-owners~find-owners', true);
+ // }
+ // self.on('showchange', removeFindOwnerFromSecondaryActions);
+ // return;
+ // }
// If context.popup API exists and popup content is small,
// use the API and set useContextPopup,