Skip to content

Commit abdcbde

Browse files
committed
Removed currentCountQuery from limitAnalytics
ref BAE-336 ref BAE-330 This was a working approach, but in order to achieve grandfathering we'll try overwriting the limit at the source in Zuul by adding a limit specific cohort with all site IDs that are currently using the feature and will be affected by the limit, so they can be grandfathered and keep using it.
1 parent 5b37d28 commit abdcbde

File tree

1 file changed

+1
-12
lines changed

1 file changed

+1
-12
lines changed

packages/limit-service/lib/config.js

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -57,17 +57,6 @@ module.exports = {
5757
formatter: count => `${count / 1000000}MB`
5858
},
5959
limitStripeConnect: {},
60-
limitAnalytics: {
61-
currentCountQuery: async (knex) => {
62-
const key = 'trafficAnalytics';
63-
const settings = await knex('settings');
64-
const setting = settings.find(s => s.key === key);
65-
const labSettings = settings.find(s => s.key === 'labs');
66-
const labsSetting = labSettings && labSettings.value && labSettings.value[key];
67-
const labsSettingEnabled = labsSetting === true;
68-
69-
return (setting && setting.value === 'true') || labsSettingEnabled;
70-
}
71-
},
60+
limitAnalytics: {},
7261
limitActivityPub: {}
7362
};

0 commit comments

Comments
 (0)